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 <[email protected]>
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 <[email protected]>, Jason Gustafson
<[email protected]>
---
.../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);
}
}