gruuya opened a new issue, #783: URL: https://github.com/apache/iceberg-rust/issues/783
Presently only `Int32` is converted into the corresponding Iceberg type `Int` https://github.com/apache/iceberg-rust/blob/42aff04658a00b390122260dbbeaf512d11af61f/crates/iceberg/src/arrow/schema.rs#L370 I'm wondering is there any downside in just adding `Int8` and `Int16` in there as well? The [spec](https://iceberg.apache.org/spec/#primitive-types) suggests that it's fine to promote a smaller into a larger type (i.e. int to long), so this might be fine? For context I'm working with a pyiceberg table, created from a parquet file with fields such as ``` optional int32 cchar (INTEGER(8,true)) = 3; optional int32 cint2 (INTEGER(16,true)) = 4; ``` and thus the corresponding Arrow schema contains the above-mentioned types. Related: https://github.com/apache/iceberg/pull/2054 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org For additional commands, e-mail: issues-h...@iceberg.apache.org
