-
Notifications
You must be signed in to change notification settings - Fork 209
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
workaround stock sqlite token count limit #4399
workaround stock sqlite token count limit #4399
Conversation
94a8606
to
ee4dbfe
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lint check:
@@ sqlite/src/parse.y @@
+ Custom tokens and rules must be defined after SQLite's.
+ Please update SQL documentation and bubble diagrams to reflect the changes you've made, if necessary.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No Wconversion errors from this change -- LGTM once the robots approve!
…d Expr op field to u16 since we now have more than 255 tokens Signed-off-by: Aakash Arayambeth <aakash10292@gmail.com>
ee4dbfe
to
739cece
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Coding style check: Success ✓.
Smoke testing: Success ✓.
Cbuild submission: Success ✓.
Regression testing: 8/532 tests failed ⚠.
The first 10 failing tests are:
sc_resume
truncatesc
schemalk_extralock_generated
truncatesc_offline_generated
comdb2sys_queueodh_generated
comdb2sys_pagesize_generated
comdb2sys
cdb2test May 2 22:45:23 2024 in-comdb2db aar_increasing_token_limit.R20240502.10 |
exclude token count check as we've hit stock sqlite limit. Also expand Expr.op field to u16 since we now have more than 255 tokens