search

apache / arrow-rs

Official Rust implementation of Apache Arrow
https://arrow.apache.org/
Apache License 2.0
2.62k stars 802 forks source link

Split `timestamp_s_to_datetime` to `date` and `time` to avoid unnecessary computation #6755

Closed jayzhan211 closed 1 day ago

jayzhan211 commented 3 days ago

Which issue does this PR close?

Closes #6746 .

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

I think we don't need to change chrono crate

tustvold commented 1 day ago

Generally code changes would be expected to have at least one review by a committer other than the author.