Issue44

Title ap_dbd_open/ap_dbd_close() might be more economic for db connection utilization
Priority wish Status resolved
Superseder Nosy List poeml
Assigned To Keywords mod_asn

Created on 2010-03-08.21:24:38 by poeml, last changed by poeml.

Messages
msg164 (view) Author: poeml Date: 2010-03-26.23:05:00
Fixed in trunk, r75.
msg146 (view) Author: poeml Date: 2010-03-09.18:16:43
This change would actually be more interesting for mod_asn, than for 
mod_mirrorbrain. mod_asn is usually configured to be used for all requests; like 
mod_geoip, it runs early enough so it can't be known if the data looked up will 
actually be used.

mod_mirrorbrain however can be configured with various exceptions to not handle a 
requests at all, and ap_dbd_acquire() runs late (not earlier than needed) and 
might not be reached at all. Especially, that stage is not reached for those 
requests that actually serve data to clients and might persist for a while 
therefore.
msg145 (view) Author: poeml Date: 2010-03-08.21:49:41
Those functions are probably much less used than ap_dbd_acquire(), and less 
tested. If they work, it'd be nice; it would be important to make sure that no 
close() is forgetten in one of the various "quick exit" code paths.
msg144 (view) Author: poeml Date: 2010-03-08.21:24:38
from the old TODO file:

* check whether usage of explicit ap_dbd_open/ap_dbd_close, instead of the
  ap_dbd_acquire wrapping function, results in better utilization of the database
  connection pool
  but apr_reslist is undergoing some fixes/changes currently (1.2.11 time), so 
let's wait
History
Date User Action Args
2010-03-26 23:05:29poemlsetstatus: testing -> resolved
2010-03-26 23:05:01poemlsetstatus: deferred -> testing
messages: + msg164
2010-03-09 18:16:43poemlsetmessages: + msg146
keyword: + mod_asn
2010-03-08 21:49:41poemlsetmessages: + msg145
2010-03-08 21:24:38poemlcreate