Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Is it possible somehow to send LIMIT and OFFSET as parameters (not using paging) ? #379

Closed
rapw3k opened this issue Sep 8, 2022 · 4 comments

Comments

@rapw3k
Copy link

rapw3k commented Sep 8, 2022

Hi @c-martinez
I would like to be able to specify the LIMIT and OFFSET of the SPARQL inside the query, not automatically via the paging option, see e.g., : https://grlc.io/api-git/ILIAD-ocean-twin/api/#/test/get_Observations_test (not working)
Is this possible somehow?
thanks

@c-martinez
Copy link
Collaborator

Hi @rapw3k,
Sorry for the delay in replying to your issue. I've had a quick look, and it appears it is not possible to do this. Basically, because having something like LIMIT ?top is invalid for SPARQL (limit should not be a SPARQL variable). Of course this would not be a problem once grlc rewrites ?_top to be a value, but because of the order in which grlc replaces variables, it runs into an issue.

@albertmeronyo -- perhaps we should have another look at the query re-writing engine? This seems to be causing a couple of issues.

@rapw3k
Copy link
Author

rapw3k commented Jan 11, 2023

thanks @c-martinez @albertmeronyo
I am coming back to this (still standing) issue.
That was exactly what i was thinking, if the user variable is replaced before the query is evaluated, then it should work in theory. Do you think it could be possible to update this in grlc ?

@c-martinez
Copy link
Collaborator

In principle I think this should be possible indeed. The real question here is on what time frame we could implement this (I know that both @albertmeronyo and I are a bit tied up with other work, but would welcome pull requests implementing these changes)

@c-martinez
Copy link
Collaborator

Haven't tested this, but I believe this should be possible since v1.3.9. I'm closing this issue, but please re-open it if it does not work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants