You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kafka.apache.org by ch...@apache.org on 2024/10/20 16:23:27 UTC

(kafka) branch trunk updated: KAFKA-17841 Use placeholders instead of string concatenation in logs (#17557)

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

chia7712 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 25e8e4cbcf3 KAFKA-17841 Use placeholders instead of string concatenation in logs (#17557)
25e8e4cbcf3 is described below

commit 25e8e4cbcf336058726848c9a5e80edb407fc2c2
Author: yx9o <ya...@163.com>
AuthorDate: Mon Oct 21 00:23:20 2024 +0800

    KAFKA-17841 Use placeholders instead of string concatenation in logs (#17557)
    
    Reviewers: Chia-Ping Tsai <ch...@gmail.com>
---
 .../src/main/java/org/apache/kafka/common/config/AbstractConfig.java    | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

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 72038b4564c..3710ab2811c 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
@@ -620,7 +620,7 @@ public class AbstractConfig {
                 provider.configure(configProperties);
                 configProviderInstances.put(entry.getKey(), provider);
             } catch (ClassNotFoundException e) {
-                log.error("Could not load config provider class " + entry.getValue(), e);
+                log.error("Could not load config provider class {}", entry.getValue(), e);
                 throw new ConfigException(providerClassProperty(entry.getKey()), entry.getValue(), "Could not load config provider class or one of its dependencies");
             }
         }