[Geoserver-devel] mysql extension

As per discussion on http://jira.codehaus.org/browse/GEOS-3685 I am creating a new mysql extension based on the new geotools datastore.

The question is what to do with the current one. Two options:

1) deprecate the old on on 2.0.x and have two extensions (naming the new one "-ng")

2) outright remove the old on 2.0.x

Opinions?

On trunk I plan to remove the old one outright.

-Justin

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

Justin Deoliveira ha scritto:

As per discussion on http://jira.codehaus.org/browse/GEOS-3685 I am creating a new mysql extension based on the new geotools datastore.

The question is what to do with the current one. Two options:

1) deprecate the old on on 2.0.x and have two extensions (naming the new one "-ng")

2) outright remove the old on 2.0.x

Opinions?

Outright remove? The old mysql datastore has not been maintained
for ages and has minimal testing.
Did you implement most of the abstract JDBC-NG tests for the
mysql version?

Cheers
Andrea

--
Andrea Aime
OpenGeo - http://opengeo.org
Expert service straight from the developers.

Andrea Aime wrote:

Justin Deoliveira ha scritto:

As per discussion on http://jira.codehaus.org/browse/GEOS-3685 I am creating a new mysql extension based on the new geotools datastore.

The question is what to do with the current one. Two options:

1) deprecate the old on on 2.0.x and have two extensions (naming the new one "-ng")

2) outright remove the old on 2.0.x

Opinions?

Outright remove? The old mysql datastore has not been maintained
for ages and has minimal testing.
Did you implement most of the abstract JDBC-NG tests for the
mysql version?

Cool, thanks. Most of the tests are implemented, 17 out of 21. The ones i missed:

3DTest
ForeignKeyTest
SkiColumnTest
ViewTest

I will throw it on my todo list to implement them. Although I probably won't implement ForeignKeyTest (actually I think we agreed to kill that functionality from jdbc-ng). 3DTest... hmmm not sure.

Cheers
Andrea

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

Justin Deoliveira ha scritto:
  > Cool, thanks. Most of the tests are implemented, 17 out of 21. The ones

i missed:

3DTest
ForeignKeyTest
SkiColumnTest

Now tell me the truth. It's winter, you're in NY, you're experiencing
ski withdrawal, don't you? :wink:

ViewTest

I will throw it on my todo list to implement them. Although I probably won't implement ForeignKeyTest (actually I think we agreed to kill that functionality from jdbc-ng). 3DTest... hmmm not sure.

Not really necessary. Would be nice to have column skipping and views
in the mix thought (just a wish, by no means a blocker).

Cheers
Andrea

--
Andrea Aime
OpenGeo - http://opengeo.org
Expert service straight from the developers.

Andrea Aime wrote:

Justin Deoliveira ha scritto:
> Cool, thanks. Most of the tests are implemented, 17 out of 21. The ones

i missed:

3DTest
ForeignKeyTest
SkiColumnTest

Now tell me the truth. It's winter, you're in NY, you're experiencing
ski withdrawal, don't you? :wink:

LOL. Yeah it appears so. It must be the fact that it is 16 degrees celcius outside at the moment :slight_smile:

ViewTest

I will throw it on my todo list to implement them. Although I probably won't implement ForeignKeyTest (actually I think we agreed to kill that functionality from jdbc-ng). 3DTest... hmmm not sure.

Not really necessary. Would be nice to have column skipping and views
in the mix thought (just a wish, by no means a blocker).

Making a jira for myself to do this.

Cheers
Andrea

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