-
Notifications
You must be signed in to change notification settings - Fork 163
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Failed to index spatial geometries with Lucene #2489
Comments
Hi, thank you for providing the snippet, this is really useful to check if we can solve the issue. EDIT: just switching to JTS 1.17.1 doesn't help, looks like a Spatial4J issue. The snippet fails in the spatial4j WKTReader facade parsing the multipolygon, but another program using (only) the JTS WKTReader seems to work just fine |
This seems to work, though it may not be a clean solution
|
Hi thank you for your reply @barthanssens and I would like to point out that this is still not supported in the main rdf4j repo, if you could check this function: https://github.com/eclipse/rdf4j/blob/f055be40e43d1b63d7a746844e5052f6bd255cec/core/sail/lucene/src/main/java/org/eclipse/rdf4j/sail/lucene/LuceneDocument.java#L143 This is happening on index time and basically it is catching an exception but the stack trace is never printed so you'd never know that something wrong happened while the log that has to appear is the following:
And basically I tried to modify this function mentioned above to use the JTS spatial context but unfortunately this lead to the error mentioned above regarding overlapping polygons:
So my question is that is there any way to tell lucene to use a JTS spatial context or to make this work out of the box without modifying the source code ? Otherwise thank you and if the snippet you provided solves the problem I will try to fix it up in the LuceneDocument class, recompile and make a pull request. |
I have to look into it, I'm not very familiar with geospatial... but if setting spatial context isn't supported yet, it's indeed a good idea to make it a bit more configurable. |
…//github.com/AlyHdr/rdf4j into eclipse-rdf4jGH-2489-geospatial-lucene-index
…//github.com/AlyHdr/rdf4j into eclipse-rdf4jGH-2489-geospatial-lucene-index Signed-off-by: AlyHdr <ali.haidar-97@hotmail.com>
…//github.com/AlyHdr/rdf4j into eclipse-rdf4jGH-2489-geospatial-lucene-index Signed-off-by: AlyHdr <ali.haidar-97@hotmail.com>
…h time Signed-off-by: AlyHdr <ali.haidar-97@hotmail.com>
Signed-off-by: AlyHdr <ali.haidar-97@hotmail.com>
Signed-off-by: AlyHdr <ali.haidar-97@hotmail.com>
…h time Signed-off-by: AlyHdr <ali.haidar-97@hotmail.com>
Signed-off-by: AlyHdr <ali.haidar-97@hotmail.com>
Signed-off-by: AlyHdr <ali.haidar-97@hotmail.com>
* Modified the LuceneIndex so that it uses the LatLonShape to index and search over data much faster than the current implementation Signed-off-by: AlyHdr <ali.haidar-97@hotmail.com> * GH-2489 supporting all WKT geometries at index and search time Signed-off-by: AlyHdr <ali.haidar-97@hotmail.com> * GH-2489 supporting formating for rdf4j Signed-off-by: AlyHdr <ali.haidar-97@hotmail.com> * GH-2489 modified a bug in lucene index search Signed-off-by: AlyHdr <ali.haidar-97@hotmail.com>
We are trying to index POLYGON and MULTIPOLYGON geometries over Lucene to run some geosparql queries on, and it is failing on some of the geometries with this exception:
And the geometry that is failing is this:
The snippet code that was used to test this index is the following:
If there is a concrete example on how to index such geometries with lucene it would be nice to have it.
The text was updated successfully, but these errors were encountered: