aihuaxu commented on code in PR #12138: URL: https://github.com/apache/iceberg/pull/12138#discussion_r1938060630
########## core/src/main/java/org/apache/iceberg/variants/VariantMetadata.java: ########## @@ -34,4 +34,20 @@ public interface VariantMetadata extends Variants.Serialized { /** Returns the size of the metadata dictionary. */ int dictionarySize(); + + static String asString(VariantMetadata metadata) { + StringBuilder builder = new StringBuilder(); + + builder.append("VariantMetadata(dict={"); + for (int i = 0; i < metadata.dictionarySize(); i += 1) { + if (i > 0) { + builder.append(", "); + } + + builder.append(i).append(" => ").append(metadata.get(i)); Review Comment: Sorry. I mean the the strings in the dictionary are the keys from the variant object. -- 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