eldenmoon commented on code in PR #49807: URL: https://github.com/apache/doris/pull/49807#discussion_r2038715953
########## fe/fe-common/src/main/java/org/apache/doris/catalog/StructField.java: ########## @@ -42,8 +42,11 @@ public class StructField { public static final String DEFAULT_FIELD_NAME = "col"; + // The struct field name should remain the same as it was defined when it was created, which stored in FE Meta + // When we query by field name, it should be case-insensitive. + // So field name "A" and "a" cannot exist in a struct at the same time public StructField(String name, Type type, String comment, boolean containsNull) { - this.name = name.toLowerCase(); Review Comment: it this meta change compatible ? -- 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: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org