tests: reduce nested type/UDT depth to 12 for new CQL nesting limit#898
Conversation
Scylla now caps the nesting depth of CQL expressions in the
parser, rejecting deeply nested literals with:
SyntaxException code=2000 'expression nested too deeply'
Cap the deepest case at 12, the maximum depth the server now allows.
Caused by scylladb/scylladb commit e35c388 ('cql3: limit nesting depth
of function calls and CASTs in CQL parser') scylladb/scylladb@c27e322.
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThis PR adjusts test constants for nested type coverage to align with Scylla's CQL expression nesting depth limit of 12. The nested tuple test changes its maximum depth from 32 to 12 (updating schema columns and iteration range) and adds a comment documenting the constraint. Three nested UDT tests similarly reduce their maximum nesting depth parameters from 16 to 12. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. 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 |
Scylla now caps the nesting depth of CQL expressions in the parser, rejecting deeply nested literals with:
SyntaxException code=2000 'expression nested too deeply'Cap the deepest case at 12, the maximum depth the server now allows.
Caused by scylladb/scylladb@c27e322.
Pre-review checklist
./docs/source/.Fixes:annotations to PR description.