Tishj opened a new issue, #13498: URL: https://github.com/apache/iceberg/issues/13498
### Apache Iceberg version 1.9.1 (latest release) ### Query engine None ### Please describe the bug 🐞 As defined by the [VariantEncoding](https://github.com/apache/parquet-format/blob/master/VariantEncoding.md) spec, the top-level Variant group should be annotated with the `VARIANT` logical type. > The Variant group must be annotated with the VARIANT logical type. Currently the Variant group is identified using the iceberg type, because it isn't written with a `LogicalTypeAnnotation` attached: ```java } else { // if not a primitive, the typeId must be a group GroupType group = type.asGroupType(); LogicalTypeAnnotation annotation = group.getLogicalTypeAnnotation(); if (annotation instanceof ListLogicalTypeAnnotation) { return visitList(iType, group, visitor); } else if (annotation instanceof MapLogicalTypeAnnotation) { return visitMap(iType, group, visitor); } else if (iType != null && iType.isVariantType()) { // when Parquet has a VARIANT logical type, use it here return visitVariant(iType.asVariantType(), group, visitor); } ``` Parquet has a VARIANT logical type, it was introduced in this PR: https://github.com/apache/parquet-format/pull/460 ### Willingness to contribute - [ ] I can contribute a fix for this bug independently - [ ] I would be willing to contribute a fix for this bug with guidance from the Iceberg community - [x] I cannot contribute a fix for this bug at this time -- 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
