Rob Atkinson ha scritto: > I agree with this - but how to go about implementing it? > > 1) we dont want to duplicate this logic (choosing or using a JNDI source > or creating a new one) - so it probably belongs in JDBCDataStore
I'd say, in the JDCBDataStoreFactory (and its subclasses) > 2) again, we want to minimise dependencies on multiple implementations > (surely?) - so what implementation of pooling are we using and do we > want to use going forward? Use SPI lookup to find connection pool managers and instruct people to drop the right jars in the mix > 3) it probably needs someone quite familiar with both JNDI and the state > of the art pooling implementations to take this on (so I probably need > to leave my own wrappers for ConnectionPoolDataSource in place) Agreed > I see informed musings at > http://geotools.codehaus.org/J2EE+and+Connection+Pools, but no JIRA task. > Should this simply have a JIRA task created - or is it some form of > serious change that needs a PMC decision first? It's a serious change that needs a PMC decision first, and I still don't see a finalized design, just some good but general ideas. Moreover, who's gonna do the implementation? There is quite a bit of work to be done, even if we want to start with a reference jdbc datastore and have the others follow at their leisure (instead of doing a big bang change, which would be preferrable, but would require more time and someone that has all the dbs installed). > I'm also not sure or the status of unsupported/oracle vs > unsupported/oracle-spatial > - if I was to update one to use org.opengis.filter, which should I attack? Hmmm... afaik oracle-spatial was an attempt made to fix Oracle ds that was left there half changed. We probably need to remove it? If one wants it, we can always check it out from a 2.4.0-Mx tag, no? Cheers Andrea ------------------------------------------------------------------------- This SF.net email is sponsored by DB2 Express Download DB2 Express C - the FREE version of DB2 express and take control of your XML. No limits. Just data. Click to get it now. http://sourceforge.net/powerbar/db2/ _______________________________________________ Geotools-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/geotools-devel
