lasdf1234 opened a new pull request, #14325:
URL: https://github.com/apache/gravitino/pull/14325

   ### What changes were proposed in this pull request?
   
   - For **catalog/schema**-scoped maintenance policy expand, call 
`internalSchemaDispatcher` / `internalTableDispatcher` `listSchemas` / 
`listTables` then `loadSchema` / `loadTable` so missing entities are imported 
via the existing dispatcher import path.
   - Fall back to `SchemaMetaService` / `TableMetaService` listing when 
dispatchers are unavailable or listing fails.
   - Direct **table**-tagged attachments still use the stored table id 
(unchanged).
   - Unit test covering catalog-dispatcher discovery of a table absent from 
meta.
   
   ### Why are the changes needed?
   
   Expand previously only saw tables already in Gravitino `table_meta`. Iceberg 
tables under a catalog/schema with a maintenance policy but never registered 
were skipped.
   
   Fixes #14324
   
   ### Does this PR introduce _any_ user-facing change?
   
   No new config. Catalog/schema policy expand may import previously 
unregistered Iceberg schemas/tables into the entity store as a side effect of 
`load*`.
   
   ### How was this patch tested?
   
   - `TestNearestPolicyTableEnumerator` (including 
`testSchemaExpansionImportsTableViaCatalogDispatcher`)
   
   ### Non-goals
   
   - Does **not** borrow aux IRC1 `CatalogWrapperManager` in-process; lakehouse 
backends stay self-owned (jdbc/hive; rest remains HTTP).
   
   Made with [Cursor](https://cursor.com)


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

Reply via email to