betodealmeida opened a new pull request, #28317: URL: https://github.com/apache/superset/pull/28317
<!--- Please write the PR title following the conventions at https://www.conventionalcommits.org/en/v1.0.0/ Example: fix(dashboard): load charts correctly --> ### SUMMARY <!--- Describe the change below, including rationale and design decisions --> Continuing the work on https://github.com/apache/superset/issues/22862. This PR: - Adds support to catalog to the Postgres DB engine spec. - Adds a new endpoint `/catalogs/` under the DB API, to list catalogs. - Adds an optional parameter `catalog` to the `/schemas/` DB API, to specify a non-default catalog. - Updates the security manager and model to manager catalog-level permissions (automatically creating, renaming, and deleting them, similar to how we do for for schemas). Note that these are all backend changes that shouldn't affect the current behavior of Superset. In the next PR I'll start working on the UI to expose the catalog functionality, and enabling support for multi-catalog in a given database will be optional and default to false. ### BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF <!--- Skip this if not applicable --> <img width="650" alt="Screen Shot 2024-05-01 at 1 34 33 PM" src="https://github.com/apache/superset/assets/1534870/57f1281e-21ac-4a7c-a3a3-5ee2b0ecec65";> ### TESTING INSTRUCTIONS <!--- Required! What steps can be taken to manually verify the changes? --> I've updated existing tests, and I'm adding new tests for all the changes. The PR is draft so it's easier to track which changes have been covered with tests. ### ADDITIONAL INFORMATION <!--- Check any relevant boxes with "x" --> <!--- HINT: Include "Fixes #nnn" if you are fixing an existing issue --> - [ ] Has associated issue: - [ ] Required feature flags: - [ ] Changes UI - [ ] Includes DB Migration (follow approval process in [SIP-59](https://github.com/apache/superset/issues/13351)) - [ ] Migration is atomic, supports rollback & is backwards-compatible - [ ] Confirm DB migration upgrade and downgrade tested - [ ] Runtime estimates and downtime expectations provided - [ ] Introduces new feature or API - [ ] Removes existing feature or API -- 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]
