ZPublisher publishing retry policy should be configurable

Bug #143013 reported by Chris McDonough
4
Affects Status Importance Assigned to Milestone
Zope 2
Fix Released
Medium
Tres Seaver

Bug Description

Currently, Zope retries a request at most exactly 3 times before giving up. The need to retry a request is always? due to a ConflictError. In some high-write-concurrency non-performance-critical applications, it would be useful to allow administrators to be able to set this number higher. We should un-hard-code this policy and make it configurable via a config file option.

Tags: feature zope
Revision history for this message
Andreas Jung (ajung) wrote :

Would be interesting for Zope 2.8. On the other hand I don't know how important this issue will be if we have MVCC in Zope 2.8

Revision history for this message
Chris McDonough (chrism-plope) wrote :

MVCC only prevents ReadConflictErrors, normal ConflictErrors (which happen at commit time) still exist.

Revision history for this message
Ben Liles (bliles) wrote :

3 years and no progress on this?

Revision history for this message
Benji York (benji) wrote : Re: [Bug 143013] Re: ZPublisher publishing retry policy should be configurable

Ben Liles wrote:
> 3 years and no progress on this?

Your patch must have gotten lost.

Revision history for this message
Ben Liles (bliles) wrote :

Here is a patch for this that works with Zope 2.10.5.

Revision history for this message
Ben Liles (bliles) wrote :

Oops, wrong patch submitted earlier.

Revision history for this message
Ben Liles (bliles) wrote :

I have attached a patch that takes care of this.

Changed in zope2:
status: New → Fix Released
Ben Liles (bliles)
Changed in zope2:
assignee: nobody → bliles
assignee: bliles → nobody
Revision history for this message
Andreas Jung (ajung) wrote :

No ides why this bug is in stage "fix released".

The patch does not apply the current trunk and I should have a test.

Changed in zope2:
status: Fix Released → New
Changed in zope2:
importance: Medium → Wishlist
Revision history for this message
Tres Seaver (tseaver) wrote :

Note that long-term, we mean to be able to run Zope by default under a WSGI server, where this feature can be handled by middleware such as repoze.retry:

  http://pypi.python.org/pypi/repoze.retry

In the near term, I will merge your patch for the Zope 2.12 branch and the trunk.

Changed in zope2:
assignee: nobody → Tres Seaver (tseaver)
importance: Wishlist → Medium
status: New → Confirmed
Tres Seaver (tseaver)
Changed in zope2:
status: Confirmed → Fix Committed
Changed in zope2:
milestone: none → 2.12.5
Changed in zope2:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.