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

jgus pushed a commit to branch trunk
in repository https://gitbox.apache.org/repos/asf/kafka.git


The following commit(s) were added to refs/heads/trunk by this push:
     new eee2bf9686d KAFKA-6793: Unused configuration logging appears to be 
noisy and unnecessary  (#13225)
eee2bf9686d is described below

commit eee2bf9686db85514c474732874d14456ae96ebc
Author: Philip Nee <p...@confluent.io>
AuthorDate: Mon Feb 13 09:27:55 2023 -0800

    KAFKA-6793: Unused configuration logging appears to be noisy and 
unnecessary  (#13225)
    
    Warnings about unused configs are most often spurious. This patch changes 
the current warning to an info message.
    
    Reviewers: Chris Egerton <chr...@aiven.io>, Jason Gustafson 
<ja...@confluent.io>
---
 .../main/java/org/apache/kafka/common/config/AbstractConfig.java    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/clients/src/main/java/org/apache/kafka/common/config/AbstractConfig.java 
b/clients/src/main/java/org/apache/kafka/common/config/AbstractConfig.java
index e620f18f7d8..faf174034a8 100644
--- a/clients/src/main/java/org/apache/kafka/common/config/AbstractConfig.java
+++ b/clients/src/main/java/org/apache/kafka/common/config/AbstractConfig.java
@@ -18,11 +18,11 @@ package org.apache.kafka.common.config;
 
 import org.apache.kafka.common.Configurable;
 import org.apache.kafka.common.KafkaException;
+import org.apache.kafka.common.config.provider.ConfigProvider;
 import org.apache.kafka.common.config.types.Password;
 import org.apache.kafka.common.utils.Utils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-import org.apache.kafka.common.config.provider.ConfigProvider;
 
 import java.util.ArrayList;
 import java.util.Collections;
@@ -370,12 +370,12 @@ public class AbstractConfig {
     }
 
     /**
-     * Log warnings for any unused configurations
+     * Info level log for any unused configurations
      */
     public void logUnused() {
         Set<String> unusedkeys = unused();
         if (!unusedkeys.isEmpty()) {
-            log.warn("These configurations '{}' were supplied but are not used 
yet.", unusedkeys);
+            log.info("These configurations '{}' were supplied but are not used 
yet.", unusedkeys);
         }
     }
 

Reply via email to