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
Remove octal issues when parsing with input_format_values_interpret_expressions=0
#59403
Conversation
This is an automated comment for commit 419545e with description of existing statuses. It's updated for the latest CI running ❌ Click here to open a full report in a separate page Successful checks
|
@yariks5s, restarting CI doesn't make sense in most cases. Even if failures are unrelated, they are still worth attention. Especially we should not ignore AST Fuzzer and Stress tests when they find bugs, even if these bugs are unrelated to changes in the PR: |
Fixes #54380
V2 of #55575
In most cases the octals are being dealt with properly (i.e. numbers are being parsed as non-octals - in
ExpressionElementParsers
). This addresses a use-case inConstantExpressionTemplate
(which happens withSET input_format_values_interpret_expressions=0
)Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Do not interpret numbers with leading zeroes as octals when
input_format_values_interpret_expressions=0