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));


Reply via email to