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

vy pushed a commit to branch release-2.x
in repository https://gitbox.apache.org/repos/asf/logging-log4j2.git


The following commit(s) were added to refs/heads/release-2.x by this push:
     new 6c69f8b  LOG4J2-3175 Avoid KafkaManager override when topics differ. 
(#590)
6c69f8b is described below

commit 6c69f8b19a7ebb7f66bc9b8f946359d62deaa5b8
Author: foryasuaki <[email protected]>
AuthorDate: Tue Oct 19 00:52:31 2021 +0800

    LOG4J2-3175 Avoid KafkaManager override when topics differ. (#590)
---
 .../org/apache/logging/log4j/core/appender/mom/kafka/KafkaManager.java | 1 +
 src/changes/changes.xml                                                | 3 +++
 2 files changed, 4 insertions(+)

diff --git 
a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/mom/kafka/KafkaManager.java
 
b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/mom/kafka/KafkaManager.java
index 85cbedf..cd37d75 100644
--- 
a/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/mom/kafka/KafkaManager.java
+++ 
b/log4j-core/src/main/java/org/apache/logging/log4j/core/appender/mom/kafka/KafkaManager.java
@@ -143,6 +143,7 @@ public class KafkaManager extends AbstractManager {
        public static KafkaManager getManager(final LoggerContext 
loggerContext, final String name, final String topic,
                        final boolean syncSend, final Property[] properties, 
final String key) {
                StringBuilder sb = new StringBuilder(name);
+               sb.append(" ").append(topic).append(" ").append(syncSend + "");
                for (Property prop : properties) {
                        sb.append(" 
").append(prop.getName()).append("=").append(prop.getValue());
                }
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
index b0a3558..57a0d38 100644
--- a/src/changes/changes.xml
+++ b/src/changes/changes.xml
@@ -5453,6 +5453,9 @@
         Make sure all application facing log methods use their own FQCN. This 
patch
         resolves a unit test failure for the %C pattern when using the 
Category logger.
       </action>
+      <action issue="LOG4J2-2978" dev="vy" type="fix" due-to="wuqian0808">
+        Avoid KafkaManager override when topics differ.
+      </action>
     </release>
   </body>
 </document>

Reply via email to