Vilnis Termanis

> There are no tests covering the assembler change nor the functionality change. I did look to see what there was, but like you say, the assembler part is not...

> at what scale have you used this? Does that usage include containment queries? @afs , we've only used the `geof:(distance|sfWithin|sfContains)` functions so far, the latter two with `geof:buffer` only....

(sorry, one more Q @LorenzBuehmann ) > we have to live with the Quadtree for read-write Datasets. What would it mean for persistence? (From my understanding the current `STRtree` index...

> * Doesn't that [setting the context to an illegal value] cause warnings in the Fuseki log? (copied from [thread reply](https://markmail.org/message/7xj7c3ybg5qushhs)): Yes it does - three (expected) warnings from TextQueryPF:...

Apologies - I updated the description since it had a mistake. The expected behaviour section has changed to: > Current behaviour (pre-patch): > > - jena:text is exposed only with...

(Changing to draft as suggested, since the best way to address this is still being debated.)

@afs thanks for the heads-up, do you mean: - https://github.com/apache/jena/pull/1444 Is that the main one or other there others in particular? (Though it'll probably be clear when I rebase what's...

> I believe you can use the geof: functions without wrapping in a GeoDS so you don't need to add this option :-) That's a good idea @SimonBin - but...

> The simplest way to fix the tests is to not call /setrecursionlimit/, just getrecursionlimit. It's not that - there's a good reason why the tests set the limit previously:...

Only the RE Tests suite fails - it's not able to find an (internal?) docker image. I believe there's nothing I can do about that.