scs-eb commented on issue #3473: URL: https://github.com/apache/jena/issues/3473#issuecomment-3358072630
I'm still working on completing the homework I mentioned in my previous post, mainly to clarify my understanding. So I don't have much to add, however, I wanted to reply to some of the comments : > Simply adding `geo:Geometry` types to your input data makes you query work immediately (no restart or spatial index needed): > I created https://github.com/apache/jena/pull/3480 that would make your reported query work without the need to add extra geometry types or enable inferencing. This is great news and wow that was quick, I was still reading the comments when I saw that a PR was created. Really good job! 🥇 > Also, the way your query is executed does not use the spatial index at all: 🤦 I was sure it was using it because of the restart. I will add this as an homework on my end to understand when the spatial index is involved in a query, I believe this will be useful later on. > A possible small bug I noticed in your screencast: The checkbox Replace index with selected graphs is checked, but no graph is selected [...] 🤦 Looks like a misunderstanding from my end. I checked it believing this would be better to replace the initial empty spatial index. It didn't impact the indexing though, but this is not planned for this usage as I could see by your example. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
