jerryshao opened a new issue, #11137: URL: https://github.com/apache/gravitino/issues/11137
## Describe the subtask `createTable` and `loadTable` in `CatalogWrapperForREST` have federation-specific internal paths (`createTableInternal`, `loadTableInternal`) that extract FileIO properties from the remote catalog's `table.io()` and inject filtered credentials into the response. `registerTable` has no equivalent federation-specific path and falls through to the parent class, returning a `LoadTableResponse` without those FileIO-derived properties and credential fields. **Files:** - `iceberg/iceberg-rest-server/src/main/java/org/apache/gravitino/iceberg/service/CatalogWrapperForREST.java` **Fix:** Add a `registerTableInternal` method that mirrors `loadTableInternal`'s FileIO-extraction pattern. Add an `if (catalog instanceof RESTCatalog)` short-circuit in a `registerTable` override in `CatalogWrapperForREST`. Note: Issue #11138 (refactoring to `FederatedCatalogWrapper`) is the cleaner long-term fix that would obviate the short-circuit approach. ## Parent issue https://github.com/apache/gravitino/issues/11131 -- 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]
