[GitHub] jena issue #205: JENA-1277: don't use sorting in spatial queries, for much b...

2017-02-02 Thread rvesse
Github user rvesse commented on the issue: https://github.com/apache/jena/pull/205 We should already have support for most of the trigonometric functions built into ARQ as extension functions -

[GitHub] jena issue #205: JENA-1277: don't use sorting in spatial queries, for much b...

2017-01-17 Thread ajs6f
Github user ajs6f commented on the issue: https://github.com/apache/jena/pull/205 Sure, so my claims are 1) this is a good PR :) and 2) if we want to offer the kind of functionality that you are removing, that's cool, but we should do it in some appropriate, public, maintainable way

[GitHub] jena issue #205: JENA-1277: don't use sorting in spatial queries, for much b...

2017-01-17 Thread osma
Github user osma commented on the issue: https://github.com/apache/jena/pull/205 @ajs6f It takes [a bit more](http://www.movable-type.co.uk/scripts/latlong.html) than just sqrt() to calculate great-circle distances between coordinates even within the same coordinate system, e.g.

[GitHub] jena issue #205: JENA-1277: don't use sorting in spatial queries, for much b...

2017-01-17 Thread ajs6f
Github user ajs6f commented on the issue: https://github.com/apache/jena/pull/205 @osma Yeah, that's what I meant tho'-- use a calculation (via `BIND`, I guess?) of the distance (and we have [√ available to use](https://jena.apache.org/documentation/query/library-function.html))

[GitHub] jena issue #205: JENA-1277: don't use sorting in spatial queries, for much b...

2017-01-17 Thread osma
Github user osma commented on the issue: https://github.com/apache/jena/pull/205 @ajs6f I'm not sure whether you can use ORDER BY in a case like this. It would need a SPARQL function for calculating distances between coordinates, so you could say, in effect, "order by distance from

[GitHub] jena issue #205: JENA-1277: don't use sorting in spatial queries, for much b...

2017-01-13 Thread osma
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