[ZODB-Dev] Re: [Zope-Checkins] CVS: ZODB3/ZODB - Connection.py:1.88 DB.py:1.48 cPickleCache.c:1.81

Toby Dickenson tdickenson@geminidataloggers.com
Wed, 9 Apr 2003 08:16:25 +0100


On Tuesday 08 April 2003 4:56 pm, Jeremy Hylton wrote:

> +        # XXX It sucks that we have to hold the lock to read
> +        # _invalidated.  Normally, _invalidated is written by call
> +        # dict.update, which will execute atomically by virtue of the
> +        # GIL.  But some storage might generate oids where hash or
> +        # compare invokes Python code.  In that case, the GIL can't
> +        # save us.

Other parts of ZODB already assume that oids are plain strings. I think we 
discussed this for AdaptableStorage. Does that simplify things?

-- 
Toby Dickenson
http://www.geminidataloggers.com/people/tdickenson