lasdf1234 opened a new pull request, #12572: URL: https://github.com/apache/gravitino/pull/12572
### What changes were proposed in this pull request? Wire engines/aux services to the `getSecrets` API introduced in #12458: - Spark / Flink / Trino: `properties() + getSecrets()` - IRC (`DynamicIcebergConfigProvider`): standalone path merges `getSecrets()`, then overlays `JdbcCredential` so **credentials win** (aligned with JDBC connectors) - Lance: catalog/schema properties with secrets - GVFS (Java + Python): merge catalog/schema/fileset secrets into FS conf ### Why are the changes needed? Fix: #12457 Depends on: #12458 (platform `getSecrets` API). Without this PR, connectors still cannot resolve secret-URN properties omitted from default `load*`. ### Does this PR introduce _any_ user-facing change? - Yes: connector/aux runtime behavior now merges secret-manager plaintext via `getSecrets` - No new public REST/API surface beyond #12458 ### How was this patch tested? - Relies on #12458 unit coverage for `getSecrets` - Compile/review of Spark, Flink, Trino, IRC, Lance, GVFS call sites ### Note on stacked diff Until #12458 merges, GitHub may show both platform and connector commits in this PR. Review the connector commit / files under spark/flink/trino/iceberg/lance/gvfs. 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]
