Closed Bug 374024 Opened 16 years ago Closed 16 years ago

Abstract-out isolation level in Bugzilla::DB so drivers can choose their own level

Categories

(Bugzilla :: Database, enhancement)

Product:
Bugzilla
Component:
Database
Type:
enhancement
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Bugzilla 3.2

People

(Reporter: mkanat, Assigned: mkanat)

References

Details

Attachments

(1 file)

v1
1.65 KB, patch
mkanat
: review+
Details | Diff | Splinter Review 
Oracle doesn't actually support REPEATABLE READ, so we should make the isolation level a constant in Bugzilla::DB and let subclasses override it.

They should always be using REPEATABLE READ or something more strict, though, I think.
Attached patch v1 — — Splinter Review 
Okay, here we go.
Assignee: database → mkanat
Status: NEW → ASSIGNED
Attachment #258639 - Flags: review+
Checking in Bugzilla/DB.pm;
/cvsroot/mozilla/webtools/bugzilla/Bugzilla/DB.pm,v  <--  DB.pm
new revision: 1.97; previous revision: 1.96
done
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Flags: approval+
Resolution: --- → FIXED
(In reply to comment #0)

We may take SERIALIZABLE the default isolation level for Oracle.
SERIALIZABLE is more strict than REPEATABLE READ, it also makes Phantom Read impossible.
You need to log in before you can comment on or make changes to this bug.