mattfaltyn opened a new pull request, #5147: URL: https://github.com/apache/polaris/pull/5147
## Summary `setup export` previously called `list_namespaces` only at the catalog root, so its output omitted every nested namespace and every policy definition stored below the top level. This made successful exports incomplete and prevented an export/apply round trip from reconstructing the full setup. This change: - reuses the existing namespace-only `crawl_namespace` traversal for both namespace and policy export; - adds regression coverage for a child namespace and a policy defined within it; - records the user-visible fix in the changelog. Policy attachments remain outside the scope of export, as documented by the existing CLI warning. Fixes #5146 ## Validation - `uv run pytest tests/ -q` — 163 tests and 35 subtests passed - `uv run pre-commit run --files apache_polaris/cli/command/setup.py tests/test_setup_command.py` — all hooks passed - `uv run pytest integration_tests/` against a freshly built local `apache/polaris:latest` image — 15 tests passed - `./gradlew check sourceTarball distTar distZip publishToMavenLocal -x :polaris-runtime-service:test -x :polaris-admin:test -PnoIntegrationTests --continue` — 1,904 tasks, build successful - `./gradlew :polaris-runtime-service:intTest` — build successful - `./gradlew :polaris-relational-jdbc:test` — build successful - `./gradlew :polaris-floci-aws-testcontainer:intTest` — build successful - `./gradlew format compileAll` — 976 tasks, build successful - `uv build --clear` — source distribution and wheel built successfully ## Checklist - [x] 🛡️ Don't disclose security issues! (contact [email protected]) - [x] 🔗 Clearly explained why the changes are needed, or linked related issues: Fixes #5146 - [x] 🧪 Added/updated tests with good coverage, or manually tested (and explained how) - [x] 💡 Added comments for complex logic (no new complex logic) - [x] 🧾 Updated `CHANGELOG.md` - [x] 📚 Updated documentation in `site/content/in-dev/unreleased` (not needed; behavior now matches the complete-export contract) ## AI assistance Significant AI assistance was used to investigate, implement, and validate this change. I reviewed and own the final result. -- 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]
