-
Notifications
You must be signed in to change notification settings - Fork 382
Add initial QuestDB support with simple SELECT generator (Fixes #457) #1277
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
base: main
Are you sure you want to change the base?
Conversation
mrigger
left a comment
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.
Thanks, this is great! We currently don't have a CI/CD test for QuestDB. Do you think it would be possible to add one first before adding new features?
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.
Seems like an accidental commit.
| errors.add("missing required column"); | ||
| errors.add("table does not exist"); | ||
| errors.add("permission denied"); | ||
| errors.add("syntax error in INSERT statement"); |
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.
Syntax errors are always unexpected, so we should fix this in the generator.
| return r.getInteger(0, 3); | ||
| case TRUNCATE: | ||
| return r.getInteger(0, 5); | ||
| case UPDATE: |
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.
I think we should only add new features once we have a CI/CD that works for a given QuestDB version.
This PR adds initial support for QuestDB in SQLancer.
Changes included:
QuestDBQueryGenerator.javato generate simple SELECT queries.QuestDBProvider.javato use the new generator.--num-threads 1) for stability during testing.Currently, only basic SELECT queries are supported. INSERT, DELETE, and more complex queries will be added in future iterations.
This addresses Issue #457 and lays the foundation for full QuestDB testing support.