On Fri, Jan 24, 2014 at 05:55:54PM -0800, milki wrote:
> >          haves = self.object_store.find_common_revisions(graph_walker)
> > +
> > +        # Deal with shallow requests separately because the haves do
> > +        # not reflect what objects are missing
> > +        if has_shallows or has_unshallows:
> > +            # Set the shallow commits in the object_store
> > +            shallow_grafts = {}
> > +            for shallow in graph_walker.shallow:
> > +                shallow_grafts[shallow] = []
> > +            self._add_graftpoints(shallow_grafts)
> 
> These graftpoints aren't native to the repository. They are the shallows that are computed as a result of the --depth parameter from a client fetch. I use grafts to pretend the repo doesn't have any more commits behind the depth. Thus, we don't want to save them to disk.
> 
> I'm not sure how this is related to reopening the repository though.
Thanks, that makes sense. I think it would be nicer to pass the grafts
to the MissingObjectFinder directly rather than modifying global
state.

I'll merge as is - thanks.

Cheers,

Jelmer