Re: [PATCH/RFC v3 0/4] Improving performance of git clean
From: Junio C Hamano <hidden>
Date: 2016-06-15 23:04:30
erik elfström [off-list ref] writes:
Ok, thanks for looking into this. I have no well founded opinions on the implementation but I do think the performance tests would be more meaningful if the setup/cleanup code could be removed from the timed section. If the community agrees on an implementation I would be happy to convert the new tests, either directly in this series or as a follow up if that is preferred.
Let's not delay the fix and do the perf thing as a follow-up series, possibly an even independent one. In other words, let's keep the topics small.
/Erik On Tue, Apr 21, 2015 at 12:14 AM, Thomas Gummerer [off-list ref] wrote:quoted
On 04/18, Erik Elfström wrote:quoted
* Still have issues in the performance tests, see comments from Thomas Gummerer on v2I've looked at the "modern" style tests again, and I don't the code churn is worth it just for using them for the performance tests. If anyone wants to take a look at the code, it's at github.com/tgummerer/git tg/perf-lib. I think adding the test_perf_setup_cleanup command would make more sense in this case. If you want I can send a patch for that.