aihuaxu commented on code in PR #12238: URL: https://github.com/apache/iceberg/pull/12238#discussion_r1966427898
########## core/src/main/java/org/apache/iceberg/avro/TypeToSchema.java: ########## @@ -187,6 +187,21 @@ public Schema map(Types.MapType map, Schema keySchema, Schema valueSchema) { return mapSchema; } + @Override + public Schema variant() { + String recordName = "r" + fieldIds.peek(); Review Comment: Let me revert it for now. Basically in RemoveIds we have the following to add "table" as the name. > static Schema removeIds(org.apache.iceberg.Schema schema) { > return AvroSchemaVisitor.visit( > AvroSchemaUtil.convert(schema.asStruct(), "table"), new RemoveIds()); > } -- 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 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