Hi,
Yes, this issue was reported last week as well; it should be fixed in trunk.
Regards,
Thomas
On Tue, Apr 7, 2015 at 12:11 PM, John McMahon <johnnmcmaho...@gmail.com
<javascript:_e(%7B%7D,'cvml','johnnmcmaho...@gmail.com');>> wrote:
> We are experiencing null pointer exceptions when we attempt to store a
> null geometry in H2 when we close the connection
>
> Attached is a small test.
>
> org.h2.jdbc.JdbcSQLException: General error:
> "java.lang.NullPointerException" [50000-186]
> at org.h2.message.DbException.getJdbcSQLException(DbException.java:345)
> at org.h2.message.DbException.get(DbException.java:168)
> at org.h2.message.DbException.convert(DbException.java:295)
> at org.h2.message.DbException.toSQLException(DbException.java:268)
> at org.h2.message.TraceObject.logAndConvert(TraceObject.java:352)
> at org.h2.jdbc.JdbcConnection.close(JdbcConnection.java:392)
> at com.intergraph.test.NullGeometryTest.startTest(NullGeometryTest.java:46)
> at com.intergraph.test.NullGeometryTest.main(NullGeometryTest.java:21)
> Caused by: java.lang.NullPointerException
> at org.h2.mvstore.rtree.SpatialDataType.write(SpatialDataType.java:89)
> at org.h2.mvstore.rtree.SpatialDataType.write(SpatialDataType.java:80)
> at org.h2.mvstore.Page.write(Page.java:735)
> at org.h2.mvstore.Page.writeUnsavedRecursive(Page.java:818)
> at org.h2.mvstore.MVStore.storeNowTry(MVStore.java:1057)
> at org.h2.mvstore.MVStore.storeNow(MVStore.java:973)
> at org.h2.mvstore.MVStore.commitAndSave(MVStore.java:962)
> at org.h2.mvstore.MVStore.commit(MVStore.java:923)
> at org.h2.mvstore.db.TransactionStore.close(TransactionStore.java:217)
> at org.h2.mvstore.db.MVTableEngine$Store.close(MVTableEngine.java:346)
> at org.h2.engine.Database.closeOpenFilesAndUnlock(Database.java:1359)
> at org.h2.engine.Database.close(Database.java:1259)
> at org.h2.engine.Database.removeSession(Database.java:1143)
> at org.h2.engine.Session.close(Session.java:678)
> at org.h2.jdbc.JdbcConnection.close(JdbcConnection.java:384)
>
> We found that a simple null check wasn't sufficient as it messes up the
> index.
>
> We instead added a line of code that created a dummy SpatialKey when the
> given key is null. This works for now but isn't a perfect solution.
>
> @Override
> public void write(WriteBuffer buff, Object obj) {
> SpatialKey k = (SpatialKey) obj;
> int flags = 0;
> for (int i = 0; i < dimensions; i++) {
> if (k.min(i) == k.max(i)) {
> flags |= 1 << i;
> }
> }
> buff.putVarInt(flags);
> for (int i = 0; i < dimensions; i++) {
> buff.putFloat(k.min(i));
> if ((flags & (1 << i)) == 0) {
> buff.putFloat(k.max(i));
> }
> }
> buff.putVarLong(k.getId());
> }
>
>
> @Override
> public void write(WriteBuffer buff, Object obj) {
> if ( obj == null ){
> obj = new SpatialKey(0,0,0,0,0);
> }
> SpatialKey k = (SpatialKey) obj;
> int flags = 0;
> for (int i = 0; i < dimensions; i++) {
> if (k.min(i) == k.max(i)) {
> flags |= 1 << i;
> }
> }
> buff.putVarInt(flags);
> for (int i = 0; i < dimensions; i++) {
> buff.putFloat(k.min(i));
> if ((flags & (1 << i)) == 0) {
> buff.putFloat(k.max(i));
> }
> }
> buff.putVarLong(k.getId());
> }
>
> --
> 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 h2-database+unsubscr...@googlegroups.com
> <javascript:_e(%7B%7D,'cvml','h2-database%2bunsubscr...@googlegroups.com');>
> .
> To post to this group, send email to h2-database@googlegroups.com
> <javascript:_e(%7B%7D,'cvml','h2-database@googlegroups.com');>.
> Visit this group at http://groups.google.com/group/h2-database.
> For more options, visit https://groups.google.com/d/optout.
>
--
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 h2-database+unsubscr...@googlegroups.com.
To post to this group, send email to h2-database@googlegroups.com.
Visit this group at http://groups.google.com/group/h2-database.
For more options, visit https://groups.google.com/d/optout.
