Merge lp://qastaging/~jtv/maas/isolation into lp://qastaging/~maas-committers/maas/trunk
Proposed by
Jeroen T. Vermeulen
| Status: | Merged | ||||
|---|---|---|---|---|---|
| Approved by: | Jeroen T. Vermeulen | ||||
| Approved revision: | no longer in the source branch. | ||||
| Merged at revision: | 2306 | ||||
| Proposed branch: | lp://qastaging/~jtv/maas/isolation | ||||
| Merge into: | lp://qastaging/~maas-committers/maas/trunk | ||||
| Diff against target: |
47 lines (+10/-2) 2 files modified
src/maas/development.py (+5/-1) src/maas/settings.py (+5/-1) |
||||
| To merge this branch: | bzr merge lp://qastaging/~jtv/maas/isolation | ||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Raphaël Badin (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Raise isolation level from the default Read Committed to Serializable. Django 1.6 added support. In situations where previously you'd get potentially incorrect updates, you will now get an OperationalError..
Description of the change
There isn't much to test; it's a policy setting. But Ryan Harper has run this through a mild stress test and found no problems.
Jeroen
To post a comment you must log in.
Revision history for this message
| Raphaël Badin (rvb) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
