Justin Deoliveira ha scritto: >> I'm +1, but there are a few little things that are not fully ok and >> that I hope we can sort out quickly: >> * dbtype: postgisng and oracleng. What should we do about those? >> The names were set so that we can have old and new postgis in the >> same container without creating a mess. And it is still needed today, >> e.g., if you want to have versioning and postgisng in the same >> classpath, as versioning depends on the old postgis. >> I guess a more future proof dbtype could be postgis2, oracle2 > > Good point. What do you think about the following: > > * We add a dynamic check for the old datastores in the classpath. When > they are present postgis only matches "postgisng" (or "postgis2"), and > same for oracle. When they are not present however, the datastore will > match just "postgis", as well as "postgisng". Then when the old > datastores are truly removed (2.7 i guess) we can remove the classpath > check, and have them match either "postgis", or "postgisng". > > Thoughts?
Sounds like a good plan. It will somehow crack when anybody needs to use versioning postgis since that will carry along the old postgis datastore, so the connections using "postgis" will fall back on using the old datastore. I will have to find time to port the versioning datastore to the new architecture... hmmm... may not happen anytime soon :( http://jira.codehaus.org/browse/GEOT-2598 >> * the proposal does not say what the contents of unsupported/jdbc will >> be (is it a copy of the old library/jdbc, or that is kept there >> and deprecated) > Oops, I forgot to remove that as per our agreement to merge > jdbc-ng/jdbc-core into the old jdbc module, and keep the old jdbc > classes around, just deprecating them. Docs updated. Nice, thank you Cheers Andrea -- Andrea Aime OpenGeo - http://opengeo.org Expert service straight from the developers. ------------------------------------------------------------------------------ _______________________________________________ Geotools-devel mailing list Geotools-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/geotools-devel
