Github user osma commented on the issue:

    https://github.com/apache/jena/pull/205
  
    Thanks @afs for the approval. I will wait a few more days and then merge 
this, unless there are objections.
    
    The only scenario I'm mildly concerned about is if someone performs a 
spatial query for a large area with a low limit parameter, so that the result 
list gets cut off. Currently the top K results will (or at least should) be 
from near the center of the area, but with the sorting disabled, the selection 
of the top K results will become random.
    
    I have to wonder why the sorting is so slow though. I don't know much about 
spatial data processing, but I suppose that calculating the distances of ~2500 
coordinates to a given center point, and then sorting the list of 2500 results 
by those distances, shouldn't take 20 seconds. More like 20 milliseconds...


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to