Re: "git grep" parallelism question

From: Jeff King <peff@peff.net>
Date: Apr 30, 2013, 15:59 UTC
Message-ID: <20130430155939.GA31881@sigill.intra.peff.net>
In-Reply-To: <20130430080848.GP472@serenity.lan>

On Tue, Apr 30, 2013 at 09:08:49AM +0100, John Keeping wrote:

Show 18 quoted lines

> > With your patch, doesn't "tXXXX-*.sh --root $there" automatically
> > use the fast $there temporary location as the result depot, too?
> 
> No, the current code uses:
> 
>     $TEST_OUTPUT_DIRECTORY/$root/trash\ directory.tXXXX
> 
> where we don't prepend $TEST_OUTPUT_DIRECTORY/ if $root is absolute.
> 
> > If it doesn't with the current code, shouldn't it?
> 
> I think the current behaviour is fine and the two options complement
> each other.
> 
> TEST_OUTPUT_DIRECTORY is something you set once and forget about which
> says "all of the test output should go over here", whereas --root is
> passed to a specific test and says "put your output here" but does not
> affect the result aggregation which is not specific to that test.

The original intent of "--root" (and how I use it) is to set and forget it, too, via GIT_TEST_OPTS. I intentionally didn't move test results with it, because to me the point was a pure optimization: put the trash directories on a faster disk, and leave everything else identical. With "--root", any scripts which later want to look at test-results will find them in the usual place.

Your patch updates all of the in-tree spots which look at the results, but any third-party scripts would need to take it into account, too (though I have no idea if any such scripts even exist).

I'm curious if there is a good reason to want to move the results. Some possibilities I can think of are:

  1. More optimization, as results are written to the faster filesystem.
     I doubt this is noticeable, though, as the amount of data written
     is relatively small compared to the tests themselves (which are
     constantly creating and deleting repos).

  2. You can run tests in a read-only git checkout. I'm not sure how
     useful that is, though, since you would already need to compile
     git.

  3. You could have multiple sets of test results to keep or compare.
     I'd think you'd want to keep the built versions of git around, too,
     though. Which would mean that a full checkout like git-new-workdir
     would be a much simpler way to accomplish the same thing.

So I'm not against TEST_OUTPUT_DIRECTORY as a concept, but I'm having trouble seeing how it is more useful than "--root".

> Note that setting TEST_OUTPUT_DIRECTORY in config.mak affects all tests
> no matter how you run them (via make or as ./tXXXX-yyyy.sh) whereas
> setting --root=... in GIT_TEST_OPTS only affect tests run via make.

I actually consider that a feature of "--root". When I run "make test" everything happens fast. When I run the script manually (which is usually because I'm debugging), the trash directory appears in the current directory, so I can easily investigate it. And if you are running a single test, the performance impact is usually negligible (where you really notice it is when running "make -j32 test").

-Peff