sl255051 commented on code in PR #10678:
URL: https://github.com/apache/iceberg/pull/10678#discussion_r1707698118
##########
api/src/main/java/org/apache/iceberg/types/TypeUtil.java:
##########
@@ -181,11 +181,34 @@ public static Map<Integer, String> indexQuotedNameById(
return indexer.byId();
}
+ /**
+ * Creates a mapping from lower-case field names to their corresponding
field IDs.
+ *
+ * <p>This method iterates over the fields of the provided struct and maps
each field's name
+ * (converted to lower-case) to its ID. If two fields have the same
lower-case name, an
+ * `IllegalArgumentException` is thrown.
+ *
+ * @param struct the struct type whose fields are to be indexed
+ * @return a map where the keys are lower-case field names and the values
are field IDs
+ * @throws IllegalArgumentException if two fields have the same lower-case
name
+ */
public static Map<String, Integer> indexByLowerCaseName(Types.StructType
struct) {
Map<String, Integer> indexByLowerCaseName = Maps.newHashMap();
indexByName(struct)
.forEach(
- (name, integer) ->
indexByLowerCaseName.put(name.toLowerCase(Locale.ROOT), integer));
+ (name, integer) -> {
+ String key = name.toLowerCase(Locale.ROOT);
+ Integer prevValue = indexByLowerCaseName.get(key);
Review Comment:
I've updated the code to use `Preconditions.checkArgument`
--
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]
