-
Notifications
You must be signed in to change notification settings - Fork 565
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
A Recursionerror exception is raised if the SPARQL UPDATE query string contains more than 90 triples #1336
Comments
stack trace for reference:
|
I think the problem may be related to this rule:
Which is expressed here. I think rewriting the grammer to not have recursive rules may be needed, but I'm not entirely sure about that either. |
Isn't it equivalent to something like (sorry for the poor syntax) ? |
@rchateauneu it is equivalent to that, and I have tested with that and it seems to parse successfully but I still need to do some fixes to get the rest to work. |
@rchateauneu I changed the rule to this:
Just because it reads a bit simpler, but I think it is equivalent to what you wrote and what was there before. Please have a look if you can: #1340 |
When a SPARQL UPDATE query that contains more than 90 triples is parsed, a Recursionerror is raised with a message "maximum recursion depth exceeded while calling a Python object". There is already an issue related to this bug (#481), but it was closed due to lack of example code. I will now provide the code to reproduce the bug.
The text was updated successfully, but these errors were encountered: