You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@pig.apache.org by da...@apache.org on 2011/07/20 19:14:05 UTC

svn commit: r1148855 - in /pig/trunk: CHANGES.txt src/org/apache/pig/impl/util/LogUtils.java

Author: daijy
Date: Wed Jul 20 17:14:04 2011
New Revision: 1148855

URL: http://svn.apache.org/viewvc?rev=1148855&view=rev
Log:
PIG-2027: NPE if Pig don't have permission for log file

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/src/org/apache/pig/impl/util/LogUtils.java

Modified: pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1148855&r1=1148854&r2=1148855&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Wed Jul 20 17:14:04 2011
@@ -89,6 +89,8 @@ PIG-2011: Speed up TestTypedMap.java (dv
 
 BUG FIXES
 
+PIG-2027: NPE if Pig don't have permission for log file (daijy)
+
 PIG-2171: TestScriptLanguage is broken on trunk (daijy and thejas)
 
 PIG-2172: Fix test failure for ant 1.8.x (daijy)

Modified: pig/trunk/src/org/apache/pig/impl/util/LogUtils.java
URL: http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/impl/util/LogUtils.java?rev=1148855&r1=1148854&r2=1148855&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/impl/util/LogUtils.java (original)
+++ pig/trunk/src/org/apache/pig/impl/util/LogUtils.java Wed Jul 20 17:14:04 2011
@@ -169,7 +169,8 @@ public class LogUtils {
             log.error(bs.toString());
         } finally {
             try {
-                fos.close();
+                if (fos!=null)
+                    fos.close();
             } catch (IOException e) {
             }
         }