Re: [PATCH v1] diffcore-rename: speed up register_rename_src
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Apr 20, 2017, 10:40 UTC
- Message-ID
- <alpine.DEB.2.20.1704201231350.3480@virtualbox>
- In-Reply-To
- <20170419013214.q35jarvmk5jhqdyi@sigill.intra.peff.net>
Hi Peff,
On Tue, 18 Apr 2017, Jeff King wrote:
Show 11 quoted lines
> On Tue, Apr 18, 2017 at 07:44:21PM +0000, git@jeffhostetler.com wrote: > > > From: Jeff Hostetler <jeffhost@microsoft.com> > > > > Teach register_rename_src() to see if new file pair can simply be > > appended to the rename_src[] array before performing the binary search > > to find the proper insertion point. > > I guess your perf results show some minor improvement. But I suspect > this is because your synthetic repo does not resemble the real world > very much.
Please note that the synthetic test repo was added *after* coming up with the patch, *after* performance benchmarking on a certain really big repository (it is not hard to guess what use case we are optimizing, right?).
In that light, I would like to register the fact that Jeff's performance work is trying to improve a very real world, that of more than 2,000 developers in our company [*1*].
Ciao, Johannes
Footnote *1*: https://twitter.com/GabeAul/status/846189637945110528