Merge lp://qastaging/~divmod-dev/divmod.org/batch-process-start-heuristic-2963 into lp://qastaging/divmod.org

Proposed by Jean-Paul Calderone
Status: Merged
Approved by: Tristan Seligmann
Approved revision: 2679
Merged at revision: 2694
Proposed branch: lp://qastaging/~divmod-dev/divmod.org/batch-process-start-heuristic-2963
Merge into: lp://qastaging/divmod.org
Diff against target: 272 lines (+174/-3)
3 files modified
Axiom/axiom/batch.py (+43/-3)
Axiom/axiom/iaxiom.py (+10/-0)
Axiom/axiom/test/test_batch.py (+121/-0)
To merge this branch: bzr merge lp://qastaging/~divmod-dev/divmod.org/batch-process-start-heuristic-2963
Reviewer Review Type Date Requested Status
Tristan Seligmann Approve
Review via email: mp+87279@code.qastaging.launchpad.net

Description of the change

This changes the batch processing system to start the batch process any time there are remote listeners for an item that is added. This doesn't make the behaviour strictly correct, but it improves it enough to make Quotient usable.

To post a comment you must log in.
Revision history for this message
Tristan Seligmann (mithrandi) wrote :

This seems like a reasonable fix in lieu of something more complex (and I'm not sure what that would look like). Please merge.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: