-
Notifications
You must be signed in to change notification settings - Fork 124
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
Parser raises an exception for not supported query type when query starts with a comment #163
Comments
We should probably remove the comments before the check. Didn't anticipate that but that's actually quite common, some kind of explanation what the query is doing. You mean you want to print the whole query in exception? The query can be huge (i.e. stored procedure declaration). |
I'd rather log it via |
Once #164 is merged I will publish v2.2.1 with this fix. |
Can you check also #162 - it's small and actually simplifies handling of with clauses? |
When working on index-digest's PR I have found the following issue.
An example call:
Error raised:
Additionally, we should also log the query itself that caused this exception to be raised.
The text was updated successfully, but these errors were encountered: