fallintoplace opened a new pull request, #1810: URL: https://github.com/apache/iceberg-go/pull/1810
Closes #1808 ## What changed - Expand comma-separated scheme and authority groups once when options are applied. - Use direct map lookups during orphan path normalization and prefix checks. - Make overlapping groups deterministic. The lexicographically last group wins. - Add tests and a benchmark across path and group counts. ## Why Orphan cleanup applies URI equivalence rules to every referenced and listed path. The old code scanned and split every configured group for each lookup. This changes that to one lookup. The impact is speed x amount: lookup savings are multiplied by the number of paths compared. ## Checks - `go test ./...` - `go test ./table` - `go test ./table -run '^$' -bench 'BenchmarkApplyURIEquivalence$' -benchmem -count=1`\n- `go vet ./table` -- 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]
