gitcommit90 opened a new pull request, #1294: URL: https://github.com/apache/iceberg-go/pull/1294
## Summary Remove the standalone `Stat` call before `WalkDir` in the Hadoop catalog's `DropNamespace` path. `WalkDir` already reports a missing root path, so the separate pre-check widens a check-to-use race window where another caller can delete or recreate the namespace between the two filesystem calls. ## Changes - Rely on `WalkDir` for missing namespace detection - Preserve existing behavior for a namespace path that is a file, not a directory, by checking the root `WalkDir` entry and returning `ErrNoSuchNamespace` - Keep existing `WalkDir` error handling for missing namespaces and non-empty namespaces ## Verification - `go test -v ./catalog/hadoop` — pass - `golangci-lint run --timeout=10m ./catalog/hadoop/...` — 0 issues - `git diff --check` — clean Fixes #1273 -- 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]
