method_lookup('open-ils.storage.transaction.FOO')->run($client) now problematic
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Evergreen |
Fix Released
|
High
|
Unassigned |
Bug Description
There are a handful of places in open-ils.storage that do this:
$self->
Passing $client to method_lookup->run has the effect of specifying that you want a top-level request, not a subrequest. With the recent subrequest and client time zone changes to Evergreen and OpenSRF master, the upshot is that a storage method that also does transaction management will return to the top-level client not only its results, but also '1' from each invocation of $self->
Since that idiom is no longer needed (session information which which to construct the transaction ID is now automatically passed to subrequests), it should be removed to avoid breaking clients of the following methods:
open-ils.
open-ils.
open-ils.
Evergreen master
OpenSRF master
Changed in evergreen: | |
milestone: | none → 2.12-beta |
importance: | Undecided → High |
Changed in evergreen: | |
status: | New → Confirmed |
Changed in evergreen: | |
assignee: | nobody → Bill Erickson (berick) |
Changed in evergreen: | |
status: | Fix Committed → Fix Released |
A patch is available at the tip of the user/gmcharlt/ lp1664386_ fix_db_ xact_subrequest s branch:
http:// git.evergreen- ils.org/ ?p=working/ Evergreen. git;a=shortlog; h=refs/ heads/user/ gmcharlt/ lp1664386_ fix_db_ xact_subrequest s