shtripat opened a new pull request, #647: URL: https://github.com/apache/iceberg-go/pull/647
1. Field ID Reassignment: NewMetadataWithUUID() was calling iceberg.AssignFreshSchemaIDs() which reassigned all field IDs starting from 1, breaking clients that expect their original field IDs to be preserved. 2. Error Message Format: Duplicate dialect validation error wasn't wrapped with ErrInvalidViewMetadata and lacked proper capitalization, preventing MinIO from mapping it to IllegalArgumentException. 3. Schema ID Handling: While MetadataBuilder.addSchema() already normalizes schema IDs to 0 for new views, the test expected consistent behavior across view version updates. -- 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]
