danhuawang opened a new pull request, #13464: URL: https://github.com/apache/gravitino/pull/13464
### What changes were proposed in this pull request? Update `dev/docker/trino-connectors/README.md` so the connector image docs match current multi-node Trino behavior: - Document that `gravitino.metalake` is optional (omit to load every metalake) - Expand Basic and OAuth2 examples with `gravitino.dynamic-catalog.environment-variable.*` and `gravitino.iceberg.rest-uri` - Add OAuth2 IRC passthrough (`gravitino.iceberg.rest-catalog.*`) and worker env mapping for the IRC credential - Clarify Kubernetes `envFrom` / Secret mounting for worker credentials - Mark `gravitino.metalake` optional and add `dynamic-catalog.environment-variable.` in the properties table Aligned with the enterprise connector README update in datastrato/gravitino-enterprise#2224. ### Why are the changes needed? Recent connectors strip plaintext secrets from `CREATE CATALOG` sent to workers. Without env-var mappings, workers fail with missing password / OAuth credential. Without overriding `gravitino.iceberg.rest-uri`, Iceberg queries can hit an unreachable discovered `http://...:9001` endpoint. ### Does this PR introduce _any_ user-facing change? Documentation only for the Trino connector image README. ### How was this patch tested? Docs-only. Cross-checked against the working distributed Trino connector configuration and the enterprise README update. 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]
