search

JanKaul / iceberg-rust

Rust implementation of Apache Iceberg with integration for Datafusion
Apache License 2.0
106 stars 14 forks source link

timestamp with timezone primitive type should be timestamptz #25

Closed qinix closed 7 months ago

qinix commented 7 months ago

current timestamp with timezone type is named timestampz, however which should be timestamptz per the spec.

https://github.com/JanKaul/iceberg-rust/blob/200118bb87d5ba9896234323dc7da54a0946beb6/iceberg-rust-spec/src/spec/types.rs#L70

https://iceberg.apache.org/spec/#primitive-types

JanKaul commented 7 months ago

Thanks for bringing this up, I will fix it in the next days.