Re: [RDBO] cached DBI handles

2007-10-26 Thread Perrin Harkins
Hmm, why not use DBI->connect_cached for this? - Perrin - This SF.net email is sponsored by: Splunk Inc. Still grepping through log files to find problems? Stop. Now Search log events and configuration files using AJAX and a

Re: [RDBO] cached DBI handles

2007-10-26 Thread John Siracusa
Here are my thoughts for revisions: * I'm not sure ping()ing belongs here at all, at least as an auto-called method. But if we do add it, we should use DBI's ping() method rather than trying to roll our own. (As an example, "select 1" doesn't work in every database, and sometimes even when it do

[RDBO] cached DBI handles

2007-10-26 Thread Peter Karman
In the thread here: http://thread.gmane.org/gmane.comp.lang.perl.modules.dbi.rose-db-object/2303/focus=2314 John suggested a new_or_cached() method and I agreed it was a good idea. Attached is a patch against Rose::DB svn trunk that implements that method and 2 others (ping() and cache_ttl()).