git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Derrick Stolee <stolee@gmail.com>
To: Taylor Blau <me@ttaylorr.com>, git@vger.kernel.org
Subject: Re: Alternates advertisement on GitHub
Date: Fri, 26 Jul 2019 09:18:50 -0400	[thread overview]
Message-ID: <19a0d5a9-864e-584d-d869-f15b5959beba@gmail.com> (raw)
In-Reply-To: <20190726031840.GB23056@TaylorsMBP9145.attlocal.net>

On 7/25/2019 11:18 PM, Taylor Blau wrote:
> Hi everybody,
> 
> Pushes to forks of git.git hosted on GitHub now advertise the tips of
> git.git as well as branches from your fork.
> 
> You may recall that Peff and I have sent a handful of patches to allow
> repositories to customize how they gather references to advertise from
> an alternate, and then to use those alternate tips as part of the
> connectivity check (in [1] and [2], respectively).

I'm glad to hear you deployed this so quickly after review!

> GitHub used to advertise '.have's on pushes to forked repositories, but
> hasn't done so since 2012. We aggregate data from all forks into a
> 'network.git', and expose the tips of each fork as:
> 
>   refs/remotes/<fork-id>/<refname>
> 
> Each fork lists the 'network.git' as its alternate, and thus the
> advertisement can get prohibitively large when there are many forks of a
> repository.
> 
> Michael Haggerty's work on packed refs makes finding references
> pertaining only to the root computationally efficient, and [1] makes it
> possible to filter down when computing the set of references to
> advertise. With [1], we can specify that computation exactly and only
> advertise branch tips from the root of a fork network.
> 
> We've been slowly rolling this out to a handful of repository networks,
> including forks of git.git hosted on GitHub. If you host your fork on
> GitHub, you shouldn't notice anything. Hopefully, pushes to your fork
> will result in smaller packfiles. In either case, nothing should break;
> if it does, please feel free to email me, or support@github.com.

I tested this by updating 'master' in derrickstolee/git to match gitster/git
and the pack was empty (ref update only). This makes fork management so much
simpler!

Thanks!
-Stolee

  reply	other threads:[~2019-07-26 13:18 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-07-26  3:18 Alternates advertisement on GitHub Taylor Blau
2019-07-26 13:18 ` Derrick Stolee [this message]
2019-07-26 21:33   ` Taylor Blau

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=19a0d5a9-864e-584d-d869-f15b5959beba@gmail.com \
    --to=stolee@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=me@ttaylorr.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).