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

RESOLVED FIXED in Bugzilla 3.2

Status

()

Bugzilla
Database
--
enhancement
RESOLVED FIXED
11 years ago
11 years ago

People

(Reporter: Max Kanat-Alexander, Assigned: Max Kanat-Alexander)

Tracking

Bugzilla 3.2
Dependency tree / graph
Bug Flags:
approval +

Details

Attachments

(1 attachment)

v1
1.65 KB, patch
Max Kanat-Alexander
: review+
Details | Diff | Splinter Review
(Assignee)

Description

11 years ago
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.
(Assignee)

Comment 1

11 years ago
Created attachment 258639 [details] [diff] [review]
v1

Okay, here we go.
Assignee: database → mkanat
Status: NEW → ASSIGNED
Attachment #258639 - Flags: review+
(Assignee)

Comment 2

11 years ago
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
Last Resolved: 11 years ago
Flags: approval+
Resolution: --- → FIXED

Comment 3

11 years ago
(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.