FANNG1 commented on PR #10081:
URL: https://github.com/apache/gravitino/pull/10081#issuecomment-4445909791

     One question to confirm the expected behavior: if an Iceberg catalog 
already sets credential-
     providers=s3-secret-key, the current logic will not auto-detect 
jdbc-user-password anymore. Is that
     intentional? In other words, once credential-providers is explicitly set, 
inferred providers are not
     merged.
   
     Other than this, LGTM.


-- 
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