fallintoplace opened a new pull request, #1302: URL: https://github.com/apache/iceberg-go/pull/1302
Summary - make MemFS.WalkDir match roots on path boundaries instead of raw string prefixes - normalize trailing slashes on the walk root - add regression coverage for sibling prefixes like `table2` and `table_backup` Why MemFS stored full keys and filtered them with `strings.HasPrefix(key, root)`. Walking `mem://bucket/table` could therefore include `mem://bucket/table2/...` or `mem://bucket/table_backup/...`, which is surprising for tests and in-memory cleanup flows. Testing - go test ./io -run 'TestMemIO_WalkDir' -count=1 - go test ./io -count=1 - go run github.com/golangci/golangci-lint/v2/cmd/[email protected] run --timeout=10m ./io - git diff --check -- 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]
