kevinjqliu commented on PR #915: URL: https://github.com/apache/iceberg-go/pull/915#issuecomment-4490146442
> The issue was on the ingestion side. A compaction (based on iceberg-rust) normalized everything to "int + date logical type", which caused iceberg-go to refuse to append new files with the partition column encoded as "int". thanks for the additional context! This makes sense. All the other implementations can accept Avro int and Avro date (per https://github.com/apache/iceberg/issues/16414) Iceberg-go only accepts Avro int, and this PR should allow it to also accept Avro date. Orthogonally, Iceberg-go _should_ change to write as Avro date instead of Avro int, to align with the other implementations. And we will clarify the spec regarding this, i raised a devlist thread already (https://lists.apache.org/thread/qqw5oog5swmswxqqmp693vz1rw132xb6) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
