Closed waynexia closed 1 week ago
The changes across the files focus on enhancing error handling and partitioning logic within the system. Key modifications include adding a new InvalidPartition
error variant, improving the MultiDimPartitionRule
implementation, and expanding test cases to cover invalid partition rules. These updates aim to make the system more robust and informative when dealing with partition-related errors.
Files | Change Summary |
---|---|
src/operator/src/error.rs |
Added InvalidPartition variant with associated fields and updated ErrorExt for handling it. |
src/operator/src/statement/ddl.rs |
Renamed InvalidPartitionRuleSnafu to InvalidPartitionSnafu and added MultiDimPartitionRule . |
src/partition/src/multi_dim.rs |
Refactored check method in RuleChecker struct to use instance variables directly. |
tests/cases/standalone/common/partition.sql |
Introduced invalid partition rule resulting in error for incorrect partition conditions. |
tests/cases/standalone/common/partition.result |
Reflected changes in error output for invalid partition rule with specific error messages. |
(Omitted; the changes don't significantly alter control flow functionality.)
Partitions tall, partitions wide,
New rules and checks now on our side,
Errors caught with greater might,
Ensuring data stays just right.
A journey through our every query,
Ends in results accurate, not dreary.
🌟📝
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 84.61%. Comparing base (
0fc18b6
) to head (e17b36c
). Report is 40 commits behind head on main.
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
What's changed and what's your intention?
Validate partition rule on
CREATE TABLE
Checklist
Summary by CodeRabbit
New Features
InvalidPartition
error variant, providing more specific error messages for invalid partition rules.Bug Fixes
Refactor
Tests