Closed etolbakov closed 3 months ago
The changes introduce new capabilities for handling intervals within the SQL module, notably adding support for ISO 8601 format strings. Functions have been renamed, new ones added, and the entire interval transformation logic refactored to handle both interval abbreviations and ISO 8601 formats effectively. These enhancements improve the clarity and functionality of interval processing in SQL operations.
Files | Change Summary |
---|---|
Cargo.toml |
Added the dependency iso8601 version 0.6.1 . |
src/.../expand_interval.rs |
Refactoring of interval normalization logic, added support for ISO 8601 formats, and introduced new functions for expanded capabilities. |
tests/.../interval.result , tests/.../interval.sql |
Introduced new test cases for handling and calculating intervals in ISO 8601 format. |
In code where time now spans and flows,
The intervals dance, in ISO clothes.
New paths we carve, through formats old,
With clearer names, our logic's bold.
As tests ensure, our strides hold true,
This rabbit hops, with joy for you.
🐇💻✨
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.96%. Comparing base (
bc398cf
) to head (b1ef887
).
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link
https://github.com/GreptimeTeam/greptimedb/issues/4215
What's changed and what's your intention?
Add support for the format of ISO 8601 for the interval type.
Checklist
Summary by CodeRabbit
New Features
Improvements
Tests