[PATCH v2] shelve: add a shelve extension to save/restore working changes

[Kevin Bullock]

On 27 Jun 2013, at 5:54 PM, Bryan O'Sullivan wrote:

> On Thu, Jun 13, 2013 at 6:06 AM, Pierre-Yves David <pierre-yves.david at logilab.fr> wrote:
>> This is not supposed to happen. The repoview object should just proxy
>> everything to the original localrepo object.
> 
> Every access to repo.mq results in a new mq.queue object being created and destroyed. I don't know quite why this is happening, but I can see it clearly.
>  
>> I'm not saying that repoview can't be bugged. But it should not prevent you
>> from doing what you are trying to.
> 
> Since I can't maintain any state on the repo object (because mq is deleted and recreated every time I access it), it is very definitely preventing me from doing what I am trying to do :-)
> 
> Here is what I can reconstruct so far.
> 
> When I ask for repo.mq, this ends up invoking propertycache.__get__ on a localrepo.proxycls object.
> 
> That tries to setattr the attribute, which (via inheritance) calls repoview.repoview.__setattr__. This seems to set the field on the mq.mqrepo object correctly, but next time the attribute is retrieved, the propertycache.__get__ method is called again, a new object is created as a result, and the previous one is deleted once the attribute is set again.

This is the crux of the issue: repo is a filtered repo object (a repoview). When you ask for repo.mq, Python sees that the mq property isn't set on repo. So it calls through dunder-and-decorator-land down to repo.unfiltered().mq() (the _method_ in mqrepo). Then propertycache assigns the result to the 'mq' attribute on the _unfiltered_ repo.

Then the next time you ask for repo.mq, Python sees that the mq property isn't set on repo. So it calls down through __getattr__ and propertycache... and of course ends up creating a new mq.queue, assigning it to repo.unfiltered().mq, and throwing away the old one.

So that's the bizarreness. Not quite sure yet how to fix it, but we clearly need to. Perhaps we could set up a time to talk it through via IRC or mumble?

This is exactly the sort of weirdness I feared when we introduced repoview. Le sigh.

pacem in terris / мир / शान्ति / ‎‫سَلاَم‬ / 平和
Kevin R. Bullock