On 2013-08-19 09:58, Nicolas Fortin (OrbisGIS) wrote:

The first fix, isGeometry function, a mix was done on Geometry object/class:

https://github.com/nicolas-f/h2database/commit/ac5fab9fffbf5cfd63f7e67e8ce4fa90da727dbc

This commit makes no sense at all.
- you have created a unnecessary recursive call in DataType#isGeometry
- you have broken the case where the geometry stuff is not on the classpath
- you have removed a test case from TestSpatial for no reason that I can see.


Second fix, WKB export, SRID and Z was missing:

https://github.com/nicolas-f/h2database/commit/f8df369622defb0413aa15b8e78ee35b0d461ca8

This fix assumes that Geometry will always be 3D, which means that we will no longer support 2D geometry, which does not seem like a good idea to me.


Third fix, ValueGeometry.equals, GeometryCollection throws an error when equals is used, using a work around through binary comparison:

https://github.com/nicolas-f/h2database/commit/57d141acb4a97b960f704860994a283b33e69d40

This commit also makes no sense. Why is the Geometry class throwing an IllegalArgumentException? We need more explanation here.


Fourth fix, update OSGi bundle manifest in order to optionally import JTS and export some other package of h2 to be able to write a table engine in a remote OSGi bundle.

https://github.com/nicolas-f/h2database/commit/0042bf99cc677515cee6b91d4b8723e3330507bd

I'm going to leave this for Thomas to comment on.
Historically we have been very loathe to expose more of H2's internals because we have no intention of making all of this stuff part of our "supported API".

--
You received this message because you are subscribed to the Google Groups "H2 
Database" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at http://groups.google.com/group/h2-database.
For more options, visit https://groups.google.com/groups/opt_out.

Reply via email to