[ZODB-Dev] transaction "Attempts" class

[Chris McDonough]

On Wed, 2012-03-28 at 18:37 -0400, Chris McDonough wrote:
> On Wed, 2012-03-28 at 17:06 -0400, Jim Fulton wrote:
> > On Wed, Mar 28, 2012 at 4:37 PM, Chris McDonough <chrism at plope.com> wrote:
> > > On Wed, 2012-03-28 at 14:21 -0400, Jim Fulton wrote:
> > ...
> > > A decorator for running some code in the context of a txn and retrying
> > > retryable exceptions would be nice higher level behavior. Â I'd be
> > > willing to do this work over this weekend.
> > 
> > Cool.  Don't forget the transaction note part. :)
> > Too many transactions in our apps don't have
> > notes, especially transactions that happen
> > outside of web requests.
> > 
> > > In the meantime, I think the existing attempts context manager still
> > > needs the small fix I proposed in my original message. Â Can you confirm
> > > that my understanding of the its intent seems roughly correct?
> > 
> > I didn't see a statement of intent.  I think your fix us good,
> > but the code (that I wrote and your fix) makes my head hurt. :)
> > 
> > I'd say, make some test cases for the bug and make it pass.

https://mail.zope.org/pipermail/checkins/2012-March/059068.html

Look reasonable?  If so I'll go ahead and merge it and base the
decorator code on it.

- C