fallintoplace opened a new pull request, #1483: URL: https://github.com/apache/iceberg-go/pull/1483
## Summary - accept UUID-style metadata sequence prefixes with five or more digits - preserve the existing five-digit minimum and checked integer conversion - cover the 99,999 to 100,000 boundary, gzip metadata, larger sequences, short prefixes, and overflow ## Why The five digits used by Java/PyIceberg-style metadata names are a minimum padding width. Once a table reaches sequence 100,000, the filename naturally has six digits. The previous exact-width regex ignored those files, which could make the Hadoop catalog load an older metadata version. ## Testing - `go test ./catalog/hadoop` - `go test -race ./catalog/hadoop` - `go vet ./catalog/hadoop` - repository package tests excluding `io/gocloud` -- 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]
