Draft: [DRIVER-788] Validate SELECT without FROM support - #255
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
b25bae3 to
7cb5399
Compare
There was a problem hiding this comment.
Pull request overview
Adds a new integration test fixture to validate executing SELECT statements without a FROM clause (e.g., SELECT 1, SELECT now()), including both prepared and non-prepared execution paths, aligned with DRIVER-788 / issue #254.
Changes:
- Introduces
SelectWithoutFromTestsintegration tests forSELECT 1andSELECT now(). - Verifies result metadata (column name/type) and decoded values for both literal and function selectors.
- Adds a server-capability skip path when
SELECTwithoutFROMis rejected as invalid/syntax error.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| public class SelectWithoutFromTests : SharedClusterTest | ||
| { | ||
| [Test] |
| @@ -0,0 +1,71 @@ | |||
| // | |||
Jira: https://scylladb.atlassian.net/browse/DRIVER-788
GitHub issue: #254
Parent epic: https://scylladb.atlassian.net/browse/DRIVER-782
Draft PR for validating SELECT without FROM support in this driver.
Refs #254.
Scope:
This draft currently contains an empty setup commit so work can start from a linked branch without adding placeholder source changes.