Hi Andrea,
this seems to be just the same reason a hack to close ArcSDE connection pools 
is in geoserver since ages ago, using reflection to call a close method.

see http://jira.codehaus.org/browse/GEOT-126

It seems the datastore api really needs a kind to call for explicit release of 
resources.

I would say go ahead and push by now and we can push trunk to add some 
shutdown like datastore method.

Gabriel.

On Tuesday 16 October 2007 11:49:48 am Andrea Aime wrote:
> Hi,
> I'm in kind of urgent need of a close method in jdbc
> datastores so it's possible to have the connection pool
> closed down when the datastore is about to be killed,
> without waiting for the garbage collector to run
> the finalizer in JDBC2DataStore (which does the same).
> (http://svn.geotools.org/geotools/trunk/gt/modules/library/jdbc/src/main/ja
>va/org/geotools/data/jdbc/JDBC2DataStore.java)
>
> Now, I'm inclined to add a close() method right into
> JDBC2DataStore and do an instanceof check in geoserver,
> since we definitely cannot afford changing the datastore
> api now.
>
> Can I proceed? This won't impede improving the datastores
> in the future (and having a close method is a well know
> item in the datastore api wish list).
> Cheers
> Andrea
>
>
> -------------------------------------------------------------------------
> 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 browser.
> Download your FREE copy of Splunk now >> http://get.splunk.com/
> _______________________________________________
> Geotools-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/geotools-devel



-------------------------------------------------------------------------
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 browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
Geotools-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to