Re: [PATCH v6 0/7] receive-pack: only use visible refs for connectivity check

From: Taylor Blau <me@ttaylorr.com>
Date: Nov 17, 2022, 21:24 UTC
Message-ID: <Y3amjknVMzSMN4g6@nand.local>
In-Reply-To: <Y3ZNUuyWF2gDk2mA@coredump.intra.peff.net>

On Thu, Nov 17, 2022 at 10:03:46AM -0500, Jeff King wrote:

Show 15 quoted lines

> On Thu, Nov 17, 2022 at 06:46:35AM +0100, Patrick Steinhardt wrote:
>
> > this is the sixth version of my patch series that tries to improve
> > performance of the connectivity check by only considering preexisting
> > refs as uninteresting that could actually have been advertised to the
> > client.
> >
> > There are only two changes in this version compared to v5:
> >
> >     - A fix to the test setup in commit 5/7 so that tests pass when
> >       GIT_TEST_DEFAULT_INITIAL_BRANCH_NAME=main.
> >
> >     - A typo fix in the commit message of patch 6/7.
>
> Thanks. Looking at the range diff, this seems good to me!

Yep, I concur. Let's make sure that it passes CI, too, and then start merging this down. Thanks, both.

Thanks, Taylor