LP#1415898 Fix Potential Cstore Starvation in KPAC
authorJason Stephenson <jason@sigio.com>
Thu, 29 Jan 2015 12:59:07 +0000 (07:59 -0500)
committerJason Stephenson <jason@sigio.com>
Thu, 29 Jan 2015 12:59:07 +0000 (07:59 -0500)
The getit_results function in EGKPacLoader.pm does xact_begin, which
opens a connected session to a cstore drone.  It later does xact_rollback,
which does not disconnect the session.  This leaves a drone hanging for
several seconds unable to responde to new requests.  This commit switches
to a rollback call that does disconnect the session.

Signed-off-by: Jason Stephenson <jason@sigio.com>
Open-ILS/src/perlmods/lib/OpenILS/WWW/EGKPacLoader.pm

index 0598947..32e394b 100644 (file)
@@ -222,7 +222,7 @@ sub load_getit_results {
     } else { 
         $e->xact_begin;
         $ctx->{hold} = $e->retrieve_action_hold_request($hold_id);
     } else { 
         $e->xact_begin;
         $ctx->{hold} = $e->retrieve_action_hold_request($hold_id);
-        $e->xact_rollback;
+        $e->rollback;
     }
 
     return Apache2::Const::OK;
     }
 
     return Apache2::Const::OK;