Andrea Aime wrote:
> Justin Deoliveira ha scritto:
>> Here it is:
>>
>> It was actually already started, I just had to tweak it a bit.
>>
>> http://docs.codehaus.org/display/GEOTOOLS/Next+Generation+JDBC+DataStore
>>
>> It is more or less ready for voting, which I think at this point is 
>> just a formality.
> 
> 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?

> * postgisng still has a functional regression compared to postgis,
>   lack of ability to use estimated extents:
>   http://jira.codehaus.org/browse/GEOT-2572
>   My fault, I need to cook up a patch, will try to do soon (I don't
>   consider it a blocker for graduation)
Noted. I agree probably not a blocker for graduation but we should try 
to address it sooner rather than later.

> * 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.
> 
> Cheers
> Andrea
> 
> 


-- 
Justin Deoliveira
OpenGeo - http://opengeo.org
Enterprise support for open source geospatial.

------------------------------------------------------------------------------
_______________________________________________
Geotools-devel mailing list
Geotools-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/geotools-devel

Reply via email to