[incubator-pulsar] branch master updated: Log4j2.yaml configures loggers incorrectly (#1801)

[mmerli]

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

mmerli pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-pulsar.git


The following commit(s) were added to refs/heads/master by this push:
     new e22ef77  Log4j2.yaml configures loggers incorrectly (#1801)
e22ef77 is described below

commit e22ef77c35c9412461af16113b8107d8933959b4
Author: Ivan Kelly <iv...@apache.org>
AuthorDate: Thu May 17 19:40:41 2018 +0200

    Log4j2.yaml configures loggers incorrectly (#1801)
    
    log4j2.yaml has a list of loggers. However, this is not a yaml list,
    but a yaml map, so the configuration only picks up the last one.
    
    This patch changes the loggers to a yaml list.
---
 conf/log4j2.yaml | 48 +++++++++++++++++++++++-------------------------
 1 file changed, 23 insertions(+), 25 deletions(-)

diff --git a/conf/log4j2.yaml b/conf/log4j2.yaml
index cd0ec88..d98cd49 100644
--- a/conf/log4j2.yaml
+++ b/conf/log4j2.yaml
@@ -156,20 +156,6 @@ Configuration:
 
   Loggers:
 
-    Logger:
-      name: org.apache.bookkeeper
-      level: "${sys:bk.log.level}"
-      additivity: false
-      AppenderRef:
-        - ref: BkRollingFile
-
-    Logger:
-      name: org.apache.distributedlog
-      level: "${sys:bk.log.level}"
-      additivity: false
-      AppenderRef:
-        - ref: BkRollingFile
-  
     # Default root logger configuration
     Root:
       level: info
@@ -177,17 +163,29 @@ Configuration:
       AppenderRef:
         - ref: "${sys:pulsar.log.appender}"
           level: "${sys:pulsar.log.level}"
+
+    Logger:
+      - name: org.apache.bookkeeper
+        level: "${sys:bk.log.level}"
+        additivity: false
+        AppenderRef:
+          - ref: BkRollingFile
+
+      - name: org.apache.distributedlog
+        level: "${sys:bk.log.level}"
+        additivity: false
+        AppenderRef:
+          - ref: BkRollingFile
     
     # Logger to inject filter script
-#    Logger:
-#      name: org.apache.bookkeeper.mledger.impl.ManagedLedgerImpl
-#      level: debug
-#      additivity: false
-#      AppenderRef:
-#        ref: "${sys:pulsar.log.appender}"
-#        ScriptFilter:
-#          onMatch: ACCEPT
-#          onMisMatch: DENY
-#          ScriptRef:
-#            ref: filter.js
+#     - name: org.apache.bookkeeper.mledger.impl.ManagedLedgerImpl
+#       level: debug
+#       additivity: false
+#       AppenderRef:
+#         ref: "${sys:pulsar.log.appender}"
+#         ScriptFilter:
+#           onMatch: ACCEPT
+#           onMisMatch: DENY
+#           ScriptRef:
+#             ref: filter.js
         

-- 
To stop receiving notification emails like this one, please contact
mme...@apache.org.