Quick fix to avoid the layer index to get unusable in case of tx rollback

[amorgner]

This PR will disable the caching of the previously added geometry node.

Background: If the transaction in which a geometry node is added to the db is rolled back, previousGeomNode will never be reset, making the whole index unusable because of a NotFoundException being thrown by

previousGeomNode.createRelationshipTo(geomNode, SpatialRelationshipTypes.NEXT_GEOM);

There might be better solutions, but maybe this is something to start from.

[amorgner]

Google Groups thread: https://groups.google.com/forum/?fromgroups=#!topic/neo4j/o07iBeHI8TU

[peterneubauer]

@amorgner , could you update and squash this PR so I can merge it into the current code? Happy New Year!

[amorgner]

Happy New Year to you, too!

Trying to get current spatial version running again .. seems the changes 2 months ago broke things for us here ..

[peterneubauer]

Can you get a clean build to work, with a clean build of graph-collections?

On Thu, Jan 3, 2013 at 12:08 PM, amorgner notifications@github.com wrote:

Happy New Year to you, too!

Trying to get current spatial version running again .. seems the changes 2
months ago broke things for us here ..

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/60#issuecomment-11840502.

[amorgner]

Ah, sorry, I meant the spatial search in structr doesn't work with the current spatial version.

neo4j-spatial builds fine with the current graph-collections (using 1.8.1) and doesn't show the NoSuchMethodError (as reported here: https://groups.google.com/forum/?fromgroups=#!topic/neo4j/akcZ9poS1ik)

As soon as I got this to work again, I'll have a look into this issue.

[amorgner]

After upgrading to current 0.10-SNAPSHOT I cannot reproduce the NotFoundException in any way. Closing it.

[ehx-v1]

it would be better anyway to have previousGeomNode reset by a TransactionEventHandler reacting upon rollback