rdblue commented on code in PR #17010:
URL: https://github.com/apache/iceberg/pull/17010#discussion_r3508436186
##########
api/src/main/java/org/apache/iceberg/types/TypeUtil.java:
##########
@@ -195,6 +195,19 @@ public static Map<Integer, String> indexQuotedNameById(
return indexer.byId();
}
+ /**
+ * Indexes the fields of a struct from ID to a flattened name that can be
used for stats struct
+ * field names.
+ *
+ * @param struct a struct type
+ * @return an index from field ID to short names, joined by _
+ */
+ public static Map<Integer, String> indexStatsNames(Types.StructType struct) {
Review Comment:
I think the set of callers is more limited for this and I didn't want it to
be overly long, which happens when you keep chaining conventions. I think I
like it this way?
--
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]
