rdblue commented on issue #619: Add schema validator URL: https://github.com/apache/incubator-iceberg/pull/619#issuecomment-552245568 @chenjunjiedada, ignoring the problem when the name is "element" seems to solve that case: ```java Integer existingFieldId = nameToId.put(fullName, fieldId); if (existingFieldId != null && !"element".equals(name)) { throw new ValidationException( "Invalid schema: multiple fields for name %s: %s and %s", fullName, existingFieldId, fieldId); } ``` I think that's a reasonable fix for now. I think we should also go back and fix how these indexes are produced. The field names are only used in `IndexByName`, so I think we can fix this.
---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
