Aklakan commented on PR #3033:
URL: https://github.com/apache/jena/pull/3033#issuecomment-2690884412
> Test case?
Good point - the problem is that the generated JSON is simply empty - which
silently worked but is incorrect.
Trying to add a test case reveals a rat-tail of issues - for example `JSON {
"x": FOO }` does not support IRI terms such as `<BAR>` in place of FOO - so
variable substitution breaks. Using literals raises scoping errors.
At first it seemed easier to add the missing transformation support to Jena
- but now seems its more reliable to just adapt our ld viewer to use SELECT
with `application/sparql-results+json` (with all this ugly unnesting of json
sparql results and JSON number conversion depending on the datatype ).
I set this issue to draft for now - but I seems I'll withdraw this PR.
--
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]
