On Wed, Aug 02, 2017 at 08:17:29AM -0700, Shawn Pearce wrote:
> > Just peeking at torvalds/linux, we have some objects with ~35K refs
> > pointing to them (e.g., the v2.6.11 tag).
>
> Oy. I'll bet that every occurrence winds up in its own block due to
> the layout of the namespace, and so the obj block list needs 35k
> varint pointers. That requires a larger block size if it has any
> chance of fitting into the reftable format.
>
> Another option is disable the obj table for these shared-object repos.
> Its an optional part of the format and can be omitted if the reader
> isn't likely to need to lookup by SHA-1, or is willing to pay the
> brute force cost of scanning every ref.
Yeah, sorry, I meant to write a few more paragraphs. I think refusing to
generate the object table for these repos would be OK. We don't serve
any user-facing operations out of them directly[1].
I'm also open to the argument that they're simply insane. Most of the
time we don't need them to be a real repository at all. They could exist
as a bare "objects/" directory. It's only at repack time that we
actually need to know which objects are reachable[2], so we could
do a special repack that generates the list on the fly from the child
repositories.
-Peff
[1] We actually disable the ".have" advertisements, because the cost of
showing all of the shared-storage ref tips is too high. One thing
I'd like to do is be able to advertise a subset of the alternate
refs (if you're a fork of torvalds/linux, then share _just_ the refs
from there). But with the current ref code, I can't even ask for a
subset of the refs without paying the cost to walk all of them.
That's one of the things I'd like to build on top of the mmap'd
packed-refs solution (and naturally would work with reftables, too).
[2] It's a bit more complicated than just knowing the list of reachable
objects. We also want to know which ones are reachable from which
fork, as we do some trickery to avoid creating deltas across forks.
So we really do want the whole ref-list, and not something like a
de-duped set of reachable tips. I don't think that makes a
difference for anything we're discussing here, but just a bit of
trivia in case somebody is thinking about the shared-storage problem
space.
