mikebridge opened a new pull request, #44849:
URL: https://github.com/apache/superset/pull/44849

   ### SUMMARY
   
   Third PR in the stack replacing #44808 (semantic-view metadata refresh). 
**Stacked on #44835 (which is stacked on #44834)**; please review only the top 
commit, `081d2643`. It's **default-off**.
   
   This adds the authorized, UI-independent maintenance operations on top of 
the shared metadata store:
   
   - **Refresh / clear by stored view UUID.** Callers can refresh the catalog, 
clear the catalog, or clear compatibility answers for a view's owning 
connection. Each operation is authorized against the owning connection, 
re-validates the principal, configuration and binding in a fresh session, and 
is exposed only when the provider supports refresh.
   - **Authorize before acquisition.** Chart-data, query and native-filter 
contexts are checked before any semantic column or catalog discovery, so a 
denied request does **zero** provider work. Guest, viewer and editor grants are 
preserved.
   - **Typed errors on every discovery path.** Unavailable, deadline, upstream, 
in-progress and database failures map consistently to 503, 504, 502, 409 and 
503, instead of a generic 400 or a misleading 404 (the two issues raised on 
#44808).
   - **Read-only cache diagnostics.** Inspecting catalog and compatibility 
timing never refreshes anything. Result-cache inspection reports only 
already-captured identities and returns "unsupported" for fresh requests rather 
than fetching.
   
   ### TESTING INSTRUCTIONS
   
   - Affected suites: 1,191 passed locally, with 100% line and branch coverage 
on the semantic-layer package.
   - Fresh-session concurrency tests (binding, configuration and authority 
races, including repeatable-read revocation) passed on real PostgreSQL and 
MySQL, 20 cases each. They're env-gated.
   
   ### ADDITIONAL INFORMATION
   
   - [ ] Has associated issue:
   - [x] Required feature flags: default-off configuration
   - [ ] Changes UI
   - [ ] Includes DB Migration
   - [x] Introduces new feature or API
   - [ ] Removes existing feature or API
   
   Draft: stacked on #44834 and #44835; the UI comes in the next PR.
   


-- 
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]

Reply via email to