Author: aching
Date: Tue Nov 13 02:12:55 2012
New Revision: 1408538

URL: http://svn.apache.org/viewvc?rev=1408538&view=rev
Log:
GIRAPH-422: Setting the log level of the root logger to the same level
can cause issues in Hadoop. (aching)

Modified:
    giraph/trunk/CHANGELOG
    giraph/trunk/giraph/src/main/java/org/apache/giraph/graph/GraphMapper.java

Modified: giraph/trunk/CHANGELOG
URL: 
http://svn.apache.org/viewvc/giraph/trunk/CHANGELOG?rev=1408538&r1=1408537&r2=1408538&view=diff
==============================================================================
--- giraph/trunk/CHANGELOG (original)
+++ giraph/trunk/CHANGELOG Tue Nov 13 02:12:55 2012
@@ -1,6 +1,9 @@
 Giraph Change Log
 
 Release 0.2.0 - unreleased
+  GIRAPH-422: Setting the log level of the root logger to the same level
+  can cause issues in Hadoop. (aching)
+
   GIRAPH-415: Refactor / cleanup Hadoop Counters (nitay via majakabiljo)
 
   GIRAPH-413: Make building Facebook profile easier. (nitay via aching)

Modified: 
giraph/trunk/giraph/src/main/java/org/apache/giraph/graph/GraphMapper.java
URL: 
http://svn.apache.org/viewvc/giraph/trunk/giraph/src/main/java/org/apache/giraph/graph/GraphMapper.java?rev=1408538&r1=1408537&r2=1408538&view=diff
==============================================================================
--- giraph/trunk/giraph/src/main/java/org/apache/giraph/graph/GraphMapper.java 
(original)
+++ giraph/trunk/giraph/src/main/java/org/apache/giraph/graph/GraphMapper.java 
Tue Nov 13 02:12:55 2012
@@ -319,9 +319,15 @@ public class GraphMapper<I extends Writa
 
     // Set the log level
     String logLevel = conf.getLocalLevel();
-    Logger.getRootLogger().setLevel(Level.toLevel(logLevel));
-    if (LOG.isInfoEnabled()) {
-      LOG.info("setup: Set log level to " + logLevel);
+    if (!Logger.getRootLogger().getLevel().equals(Level.toLevel(logLevel))) {
+      Logger.getRootLogger().setLevel(Level.toLevel(logLevel));
+      if (LOG.isInfoEnabled()) {
+        LOG.info("setup: Set log level to " + logLevel);
+      }
+    } else {
+      if (LOG.isInfoEnabled()) {
+        LOG.info("setup: Log level remains at " + logLevel);
+      }
     }
     // Sets pattern layout for all appenders
     if (conf.useLogThreadLayout()) {


Reply via email to