diqiu50 opened a new pull request, #13212: URL: https://github.com/apache/gravitino/pull/13212
### What changes were proposed in this pull request? - Add `CatalogConnectorAdapterProvider`, a `ServiceLoader` SPI that maps a Gravitino catalog provider to a `CatalogConnectorAdapter`. - `DefaultCatalogConnectorFactory` registers discovered providers after the built-in adapters, ignores a provider whose name is already registered, and skips one that fails to load. - `ContainerSuite` mounts every jar under `GRAVITINO_TRINO_CONNECTOR_EXTRA_LIBS` into the Trino container's Gravitino plugin directory, so an integration test can load an extension jar next to the connector. ### Why are the changes needed? Adapters shipped outside the connector jar can be registered without changing the connector. Fix: #13211 ### Does this PR introduce _any_ user-facing change? New SPI `org.apache.gravitino.trino.connector.catalog.CatalogConnectorAdapterProvider`. Existing behavior is unchanged when no provider is present. ### How was this patch tested? Added `TestDefaultCatalogConnectorFactory` with fake, duplicate and broken providers registered via `META-INF/services`; existing trino-connector unit tests pass. -- 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]
