Author: daijy
Date: Tue May 11 06:28:55 2010
New Revision: 943003
URL: http://svn.apache.org/viewvc?rev=943003&view=rev
Log:
PIG-1407: Logging starts before being configured
Modified:
hadoop/pig/trunk/CHANGES.txt
hadoop/pig/trunk/src/org/apache/pig/Main.java
Modified: hadoop/pig/trunk/CHANGES.txt
URL:
http://svn.apache.org/viewvc/hadoop/pig/trunk/CHANGES.txt?rev=943003&r1=943002&r2=943003&view=diff
==============================================================================
--- hadoop/pig/trunk/CHANGES.txt (original)
+++ hadoop/pig/trunk/CHANGES.txt Tue May 11 06:28:55 2010
@@ -54,6 +54,8 @@ PIG-1309: Map-side Cogroup (ashutoshc)
BUG FIXES
+PIG-1407: Logging starts before being configured (azaroth via daijy)
+
PIG-1391: pig unit tests leave behind files in temp directory because
MiniCluster files don't get deleted (tejas)
Modified: hadoop/pig/trunk/src/org/apache/pig/Main.java
URL:
http://svn.apache.org/viewvc/hadoop/pig/trunk/src/org/apache/pig/Main.java?rev=943003&r1=943002&r2=943003&view=diff
==============================================================================
--- hadoop/pig/trunk/src/org/apache/pig/Main.java (original)
+++ hadoop/pig/trunk/src/org/apache/pig/Main.java Tue May 11 06:28:55 2010
@@ -276,15 +276,14 @@ public static void main(String args[])
logFileName =
validateLogFile(properties.getProperty("pig.logfile"), null);
}
- if(logFileName != null) {
- log.info("Logging error messages to: " + logFileName);
- }
-
pigContext.getProperties().setProperty("pig.logfile", (logFileName ==
null? "": logFileName));
// configure logging
configureLog4J(properties, pigContext);
+ if(logFileName != null) {
+ log.info("Logging error messages to: " + logFileName);
+ }
if(optimizerRules.size() > 0) {
pigContext.getProperties().setProperty("pig.optimizer.rules",
ObjectSerializer.serialize(optimizerRules));