On Fri, Aug 05, 2016 at 05:04:27PM +0200, Duy Nguyen wrote:

> On Fri, Aug 5, 2016 at 11:28 AM, Jeff King <p...@peff.net> wrote:
> > There was discussion a long time ago about storing a common zlib
> > dictionary in the packfile and using it for all of the objects. I don't
> > recall whether there were any patches, though. It does create some
> > complications with serving clones/fetches, as they may ask for a subset
> > of the objects (so you have to send them the whole dictionary, which may
> > be a lot of overhead if you're only fetching a few objects).
> I'm nit picking since it's not actually "all objects". But pack v4
> patches have two dictionaries (or more? i don't remember) for commits
> and trees :)

I couldn't remember if that zlib stuff was part of packv4 or not. I like
many of the ideas in pack v4, but I do worry a lot about the
compatibility issues, as packv2 is the on-the-wire format.

Being able to send bytes directly off disk with minimal processing is a
key thing that makes running a large git hosting site practical. One of
the things that makes nervous is having to do on-the-fly conversion when
serving fetches and clones (but to be clear that is just gut
nervousness; I haven't done any actual testing with the proto-packv4

To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to