LorenzBuehmann opened a new pull request, #1331:
URL: https://github.com/apache/jena/pull/1331
The current implementation has some drawbacks:
* a `LIMIT` on the query is not transparently respected by SPF, as all
results are first wrapped
* while possible to set a custom `LIMIT` in a geospatial property
functions like `spatial:withinBoxGeom`, you have to find a value such that a
`LIMIT` of the outer query would be reflected in the end
* for each possible found feature based in the geospatial index
* a separate query iterator is created with a single binding
* all of which are passed to a concatenating iterator
* the closing of unconsumed `QueryIterPlainWrapper`s stalls Jena for an
extended period of time
* also that function isn't therefore lazily evaluated
The commit makes use of Java streams i) respecting still the custom limit of
the property function and ii) resulting an a single iterator being lazily
evaluated, i.e. a possible query limit will always be reflected -> no need to
use the custom limit param anymore.
--
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]
