Closed etolbakov closed 1 week ago
The recent changes enhance handling and transformation of SQL INTERVAL
data types by refining casting logic and enhancing support for varied time units like milliseconds, microseconds, and nanoseconds. The updates include modifications to the expand_interval.rs
file and interval-related SQL queries to ensure accurate interval representations and seamless transformations.
File(s) | Change Summary |
---|---|
src/sql/src/statements/transform/expand_interval.rs |
Added import for DataType , refined interval handling logic, and updated public entity declarations for better interval transformation. |
tests/cases/standalone/common/types/interval/interval.result |
Added SQL queries showcasing results with INTERVAL data type for specific time durations, demonstrating ::INTERVAL casting. |
tests/cases/standalone/common/types/interval/interval.sql |
Included new interval type casting queries such as '3 hours' and '3y2mon' to illustrate usage and results. |
In the world of SQL so bright and true,
Intervals now shine with milliseconds too.
From hours to years, the time flows free,
Cast your queries, and let time be.
With code refined, transformations sing,
Celebrating each tick of the interval's ring.
🌟
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?
Attention: Patch coverage is 99.00000%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 84.57%. Comparing base (
df0fff2
) to head (bde41da
).
Thanks @tisonkun! Please go ahead ! 🤝
I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link
https://github.com/GreptimeTeam/greptimedb/issues/4168
What's changed and what's your intention?
Checklist
Summary by CodeRabbit
New Features
Bug Fixes