search

apache / iceberg-rust

Apache Iceberg
https://rust.iceberg.apache.org/
Apache License 2.0
616 stars 138 forks source link

Fix day timestamp micro #312

Closed marvinlanhenke closed 6 months ago

marvinlanhenke commented 6 months ago

Which issue does this PR close?

Closes #311

Rationale for this change

Align timestamp conversion with Java implementation. Unblock #309

What changes are included in this PR?

Are these changes tested?

Yes. Unit tests are included.

marvinlanhenke commented 6 months ago

@liurenjie1024 as discussed in #311 PTAL.

Should we handle Year, Month and Hours accordlingly (in a different PR), if we can merge this - I can finish #309.