You are viewing a plain text version of this content. The canonical link for it is here.
Posted to common-commits@hadoop.apache.org by ey...@apache.org on 2009/02/26 06:08:49 UTC

svn commit: r748019 - /hadoop/core/trunk/src/contrib/chukwa/src/java/org/apache/hadoop/chukwa/inputtools/log4j/Log4JMetricsContext.java

Author: eyang
Date: Thu Feb 26 05:08:49 2009
New Revision: 748019

URL: http://svn.apache.org/viewvc?rev=748019&view=rev
Log:
HADOOP-5301. Fix logging of Hadoop metrics to one log file per java process.
(Jerome Boulon via eyang).

Modified:
    hadoop/core/trunk/src/contrib/chukwa/src/java/org/apache/hadoop/chukwa/inputtools/log4j/Log4JMetricsContext.java

Modified: hadoop/core/trunk/src/contrib/chukwa/src/java/org/apache/hadoop/chukwa/inputtools/log4j/Log4JMetricsContext.java
URL: http://svn.apache.org/viewvc/hadoop/core/trunk/src/contrib/chukwa/src/java/org/apache/hadoop/chukwa/inputtools/log4j/Log4JMetricsContext.java?rev=748019&r1=748018&r2=748019&view=diff
==============================================================================
--- hadoop/core/trunk/src/contrib/chukwa/src/java/org/apache/hadoop/chukwa/inputtools/log4j/Log4JMetricsContext.java (original)
+++ hadoop/core/trunk/src/contrib/chukwa/src/java/org/apache/hadoop/chukwa/inputtools/log4j/Log4JMetricsContext.java Thu Feb 26 05:08:49 2009
@@ -87,14 +87,17 @@
 				  appender.setLayout(layout);
 				  appender.setAppend(true);
 				  if(properties.getProperty("log4j.appender.chukwa."+contextName+".Dir")!=null) {
-					  String logName = properties.getProperty("log4j.appender.chukwa."+contextName+".Dir")+File.separator+"chukwa-"+user+"-"+contextName+".log";
+				    String logName = properties.getProperty("log4j.appender.chukwa."+contextName+".Dir")
+				    +File.separator+"chukwa-"+user+"-"+contextName + "-" + System.currentTimeMillis() +".log";
+
 					  // FIXME: Hack to make the log file readable by chukwa user. 
 					  if(System.getProperty("os.name").intern()=="Linux".intern()) {
 						  Runtime.getRuntime().exec("chmod 640 "+logName);
 					  }
 				      appender.setFile(logName);					  
 				  } else {
-				      appender.setFile(metricsLogDir+File.separator+"chukwa-"+user+"-"+contextName+".log");
+				    appender.setFile(metricsLogDir+File.separator+"chukwa-"+user+"-"
+				        +contextName + "-" + System.currentTimeMillis()+ ".log");
 				  }
 				  appender.activateOptions();
 				  appender.setRecordType(properties.getProperty("log4j.appender.chukwa."+contextName+".recordType"));