-
Notifications
You must be signed in to change notification settings - Fork 165
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
Query with additional whitespace between 'NOT' and 'IN' or 'NOT' and 'EXISTS' fails with parse error #3957
Comments
abrokenjester
added
🐞 bug
issue is a bug
📦 sparql
affects the SPARQL parser / engine
labels
Jun 5, 2022
Cause is that in our parser grammar, we treat |
abrokenjester
added a commit
that referenced
this issue
Jun 5, 2022
- regenerate parser from javacc using 7.0.4.
5 tasks
abrokenjester
added a commit
that referenced
this issue
Jun 5, 2022
- regenerate parser from javacc using 7.0.4.
hmottestad
added a commit
that referenced
this issue
Jun 6, 2022
…led properly by the JVM C2 compiler as evident in SPARQLParseBenchmark
hmottestad
added a commit
that referenced
this issue
Jun 6, 2022
GH-3957 split NOT IN and NOT EXISTS into separate tokens
patrickwyler
pushed a commit
to patrickwyler/rdf4j
that referenced
this issue
Jun 20, 2022
- regenerate parser from javacc using 7.0.4.
patrickwyler
pushed a commit
to patrickwyler/rdf4j
that referenced
this issue
Jun 20, 2022
… not be compiled properly by the JVM C2 compiler as evident in SPARQLParseBenchmark
patrickwyler
pushed a commit
to patrickwyler/rdf4j
that referenced
this issue
Jun 20, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Current Behavior
When parsing:
or
parsing fails with a lexical error, caused by the additional whitespace between
NOT
andIN
/EXISTS
.Expected Behavior
Parser should ignore additional whitespace between tokens, as per https://www.w3.org/TR/sparql11-query/#whitespace
Steps To Reproduce
No response
Version
4.0.1
Are you interested in contributing a solution yourself?
Yes
Anything else?
Split out from GH-3947
The text was updated successfully, but these errors were encountered: