[Neo4j] GeoSpatial LayerNodeIndex

2011-11-03 Thread b m
org.neo4j.gis.spatial.indexprovider.LayerNodeIndex#add doesn't seem to
conform to the org.neo4j.graphdb.index.Index#add interface specifications:

Adds a key/value pair for entity to the index. If that key/value pair for
the entity is already in the index it's up to the implementation to make it
so that such an operation is idempotent.

In my tests, calling LayerNodeIndex#add N number of times for the same Node
resulted in N number of nodes being added to the corresponding Spatial
Layer for that index.

https://github.com/neo4j/spatial/blob/master/src/main/java/org/neo4j/gis/spatial/indexprovider/LayerNodeIndex.java

Any thoughts?
___
Neo4j mailing list
User@lists.neo4j.org
https://lists.neo4j.org/mailman/listinfo/user


Re: [Neo4j] GeoSpatial LayerNodeIndex

2011-11-03 Thread Peter Neubauer
Mmh,
yes that is correct, thanks for spotting it. Could you raise an issue
and do you care to fork and fix
(https://github.com/neo4j/spatial/issues) ? Otherwise, we might get to
it first after next week ...

Cheers,

/peter neubauer

GTalk:      neubauer.peter
Skype       peter.neubauer
Phone       +46 704 106975
LinkedIn   http://www.linkedin.com/in/neubauer
Twitter      http://twitter.com/peterneubauer

http://www.neo4j.org              - NOSQL for the Enterprise.
http://startupbootcamp.org/    - Öresund - Innovation happens HERE.



On Thu, Nov 3, 2011 at 6:29 PM, b m bm3...@gmail.com wrote:
 org.neo4j.gis.spatial.indexprovider.LayerNodeIndex#add doesn't seem to
 conform to the org.neo4j.graphdb.index.Index#add interface specifications:

 Adds a key/value pair for entity to the index. If that key/value pair for
 the entity is already in the index it's up to the implementation to make it
 so that such an operation is idempotent.

 In my tests, calling LayerNodeIndex#add N number of times for the same Node
 resulted in N number of nodes being added to the corresponding Spatial
 Layer for that index.

 https://github.com/neo4j/spatial/blob/master/src/main/java/org/neo4j/gis/spatial/indexprovider/LayerNodeIndex.java

 Any thoughts?
 ___
 Neo4j mailing list
 User@lists.neo4j.org
 https://lists.neo4j.org/mailman/listinfo/user

___
Neo4j mailing list
User@lists.neo4j.org
https://lists.neo4j.org/mailman/listinfo/user