commit 09b27a82dd6d0c350b9a6605d958826c8aa49a80
author Edwin Kempin <edwin.kempin@sap.com> Fri Mar 13 10:32:04 2015 +0100
committer Edwin Kempin <edwin.kempin@sap.com> Fri Mar 13 10:32:04 2015 +0100
tree a828aaf97d512efc9b236226e9362e538ee23a41
parent 7d980122f1607a5d5e5c1788bfa95d9608b25d83

Allow import log to be configured through external log4j.properties

Change-Id: Ic95eb8a5e09ad145730a82eac5dd5fb1b5c72b53
Signed-off-by: Edwin Kempin <edwin.kempin@sap.com>

src/main/java/com/googlesource/gerrit/plugins/importer/ImportLog.java

diff --git a/src/main/java/com/googlesource/gerrit/plugins/importer/ImportLog.java b/src/main/java/com/googlesource/gerrit/plugins/importer/ImportLog.java
index bf1d1e9..cf0da0f 100644
--- a/src/main/java/com/googlesource/gerrit/plugins/importer/ImportLog.java
+++ b/src/main/java/com/googlesource/gerrit/plugins/importer/ImportLog.java
@@ -27,6 +27,7 @@
 import com.google.inject.Inject;
 import com.google.inject.Singleton;
 
+import org.apache.log4j.AsyncAppender;
 import org.apache.log4j.Level;
 import org.apache.log4j.LogManager;
 import org.apache.log4j.Logger;
@@ -121,9 +122,11 @@
   public void start() {
     if (!started) {
       Logger importLogger = LogManager.getLogger(IMPORT_LOG_NAME);
-      importLogger.removeAllAppenders();
-      importLogger.addAppender(systemLog.createAsyncAppender(
-          importLogger.getName(), new ImportLogLayout()));
+      String loggerName = importLogger.getName();
+      AsyncAppender asyncAppender = systemLog.createAsyncAppender(
+          loggerName, new ImportLogLayout());
+      importLogger.removeAppender(loggerName);
+      importLogger.addAppender(asyncAppender);
       importLogger.setAdditivity(false);
       started = true;
     }