You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by si...@apache.org on 2018/09/04 18:15:28 UTC

[incubator-pulsar] branch master updated: Fix log4j config to not redirect BK logs into separate file (#2482)

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

sijie 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 37f266b  Fix log4j config to not redirect BK logs into separate file (#2482)
37f266b is described below

commit 37f266bb0357fa825f1b3c2e99582526c43bba84
Author: Matteo Merli <mm...@apache.org>
AuthorDate: Tue Sep 4 11:15:24 2018 -0700

    Fix log4j config to not redirect BK logs into separate file (#2482)
    
    ### Motivation
    
    Currently all `org.apache.bookkeeper` messages printed by broker are being routed to a different log file `broker.log.bk`. Removing for now the special config for BK logs.
---
 conf/log4j2.yaml | 50 --------------------------------------------------
 1 file changed, 50 deletions(-)

diff --git a/conf/log4j2.yaml b/conf/log4j2.yaml
index ab2b9ab..45ed5d9 100644
--- a/conf/log4j2.yaml
+++ b/conf/log4j2.yaml
@@ -35,10 +35,6 @@ Configuration:
         value: "info"
       - name: "pulsar.routing.appender.default"
         value: "Console"
-      - name: "bk.log.level"
-        value: "info"
-      - name: "bk.log.appender"
-        value: "BkRollingFile"
 
   # Example: logger-filter script
   Scripts:
@@ -85,34 +81,6 @@ Configuration:
             IfLastModified:
               age: 30d
 
-    # Rolling file appender configuration for bk
-    RollingRandomAccessFile:
-      name: BkRollingFile
-      fileName: "${sys:pulsar.log.dir}/${sys:pulsar.log.file}.bk"
-      filePattern: "${sys:pulsar.log.dir}/${sys:pulsar.log.file}.bk-%d{MM-dd-yyyy}-%i.log.gz"
-      immediateFlush: true
-      PatternLayout:
-        Pattern: "%d{HH:mm:ss.SSS} [%t] %-5level %logger{36} - %msg%n"
-      Policies:
-        TimeBasedTriggeringPolicy:
-          interval: 1
-          modulate: true
-        SizeBasedTriggeringPolicy:
-          size: 1 GB
-        # Trigger every day at midnight that also scan
-        # roll-over strategy that deletes older file
-        CronTriggeringPolicy:
-          schedule: "0 0 0 * * ?"
-      # Delete file older than 30days
-      DefaultRolloverStrategy:
-          Delete:
-            basePath: ${sys:pulsar.log.dir}
-            maxDepth: 2
-            IfFileName:
-              glob: "*/${sys:pulsar.log.file}.bk*log.gz"
-            IfLastModified:
-              age: 30d
-
     # Routing
     Routing:
       name: RoutingAppender
@@ -173,24 +141,6 @@ Configuration:
         AppenderRef:
           - ref: Console
 
-      - name: org.apache.bookkeeper
-        level: "${sys:bk.log.level}"
-        additivity: false
-        AppenderRef:
-          - ref: "${sys:bk.log.appender}"
-
-      - name: org.apache.distributedlog
-        level: "${sys:bk.log.level}"
-        additivity: false
-        AppenderRef:
-          - ref: "${sys:bk.log.appender}"
-
-      - name: org.apache.zookeeper
-        level: "${sys:bk.log.level}"
-        additivity: false
-        AppenderRef:
-          - ref: "${sys:bk.log.appender}"
-
       - name: verbose
         level: info
         additivity: false