lasdf1234 opened a new pull request, #13150:
URL: https://github.com/apache/gravitino/pull/13150
### What changes were proposed in this pull request?
Cherry-pick of #13125 onto `branch-1.3`, resolving the conflicts that
blocked #13145.
Mark `jdbc-user` as not hidden in Iceberg and Paimon catalog properties
metadata, matching JDBC catalogs after #12750. Update unit tests and docs.
### Why are the changes needed?
#12750 treated `jdbc-user` as an account identifier (plaintext) for JDBC,
but Iceberg/Paimon still masked it as `******`, causing cross-catalog
inconsistency.
The automated cherry-pick (#13145) left conflict markers in docs and the
Paimon test. This PR applies the same behavior on a clean `branch-1.3` fork
branch.
Fix: #13124
Original: #13125
Replaces: #13145
### Does this PR introduce _any_ user-facing change?
Yes. `GET .../catalogs/{catalog}` for lakehouse-iceberg / lakehouse-paimon
now returns `jdbc-user` in plaintext. `jdbc-password` remains masked.
### How was this patch tested?
```
./gradlew :catalogs:catalog-lakehouse-iceberg:test --tests
org.apache.gravitino.catalog.lakehouse.iceberg.TestIcebergCatalogPropertiesMetadata
\
:catalogs:catalog-lakehouse-paimon:test --tests
org.apache.gravitino.catalog.lakehouse.paimon.TestPaimonCatalogPropertiesMetadata
-PskipITs
```
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]
