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

   ### What changes were proposed in this pull request?
   
   Enable multi-metalake mode (`gravitino.use-single-metalake=false`) for Trino 
connector versions 469-472 and 473-478 by marking them as supporting catalog 
names with metalake prefix. Disable the flag for 446-451 which does not support 
this mode.
   
   ### Why are the changes needed?
   
   Fix: #10717
   
   Connector versions 469+ and 473+ reported `supportCatalogNameWithMetalake() 
== false`, causing a hard startup error when 
`gravitino.use-single-metalake=false`. These versions do support the feature, 
so the flag is corrected and previously disabled multi-metalake tests are 
re-enabled.
   
   ### Does this PR introduce _any_ user-facing change?
   
   Yes: users on Trino 469+ with `gravitino.use-single-metalake=false` can now 
start the connector successfully. Documentation for the 
`gravitino.use-single-metalake` property has been added to the configuration 
reference.
   
   ### How was this patch tested?
   
   - Re-enabled `MultiMetalake` tests in `TestGravitinoConnector469` and 
`TestGravitinoConnector478`.
   - `./gradlew :trino-connector:trino-connector:test 
:trino-connector:trino-connector-469-472:test 
:trino-connector:trino-connector-473-478:test -PskipITs` — all 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]

Reply via email to