This is an automated email from the ASF dual-hosted git repository.

diqiu50 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/gravitino.git


The following commit(s) were added to refs/heads/main by this push:
     new d4533f6dc7 [#14342] fix(trino-connector): Stop logging connector 
configuration (#14344)
d4533f6dc7 is described below

commit d4533f6dc7c1de1d40bdcf16d3590c2594a08950
Author: Yuhui <[email protected]>
AuthorDate: Thu Oct 29 14:08:15 2026 +0800

    [#14342] fix(trino-connector): Stop logging connector configuration (#14344)
    
    ### What changes were proposed in this pull request?
    
    Remove the configuration map from the connector creation success log,
    retaining the connector name.
    
    ### Why are the changes needed?
    
    The configuration can contain plaintext credentials that should not
    appear
    in INFO logs.
    
    Fix: #14342
    
    ### Does this PR introduce _any_ user-facing change?
    
    The connector creation success log no longer includes configuration
    values.
    
    ### How was this patch tested?
    
    Passed the Trino connector module's spotlessCheck and git diff --check.
    No new tests were added for this logging-only change.
---
 .../gravitino/trino/connector/GravitinoConnectorPluginManager.java      | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/GravitinoConnectorPluginManager.java
 
b/trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/GravitinoConnectorPluginManager.java
index daaa73bfbe..6b0c26357e 100644
--- 
a/trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/GravitinoConnectorPluginManager.java
+++ 
b/trino-connector/trino-connector/src/main/java/org/apache/gravitino/trino/connector/GravitinoConnectorPluginManager.java
@@ -291,7 +291,7 @@ public class GravitinoConnectorPluginManager {
           new ThreadContextClassLoader(plugin.getClass().getClassLoader())) {
         ConnectorFactory connectorFactory = 
plugin.getConnectorFactories().iterator().next();
         Connector connector = connectorFactory.create(connectorName, config, 
context);
-        LOG.info("create connector %s with config %s successful", 
connectorName, config);
+        LOG.info("create connector %s successful", connectorName);
         return connector;
       }
     } catch (Exception e) {

Reply via email to