pig-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From prkommire...@apache.org
Subject svn commit: r1577753 - in /pig/trunk: CHANGES.txt src/org/apache/pig/PigServer.java src/org/apache/pig/tools/ToolsPigServer.java test/org/apache/pig/pigunit/pig/PigServer.java
Date Fri, 14 Mar 2014 23:04:36 GMT
Author: prkommireddi
Date: Fri Mar 14 23:04:35 2014
New Revision: 1577753

URL: http://svn.apache.org/r1577753
Log:
PIG-3811: PigServer.registerScript() wraps exception incorrectly on parsing errors

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/src/org/apache/pig/PigServer.java
    pig/trunk/src/org/apache/pig/tools/ToolsPigServer.java
    pig/trunk/test/org/apache/pig/pigunit/pig/PigServer.java

Modified: pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1577753&r1=1577752&r2=1577753&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Fri Mar 14 23:04:35 2014
@@ -99,6 +99,8 @@ OPTIMIZATIONS
  
 BUG FIXES
 
+PIG-3811: PigServer.registerScript() wraps exception incorrectly on parsing errors (prkommireddi)
+
 PIG-3806: PigServer constructor throws NPE after PIG-3765 (aniket486)
 
 PIG-3801: Auto local mode does not call storeSchema (aniket486)

Modified: pig/trunk/src/org/apache/pig/PigServer.java
URL: http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/PigServer.java?rev=1577753&r1=1577752&r2=1577753&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/PigServer.java (original)
+++ pig/trunk/src/org/apache/pig/PigServer.java Fri Mar 14 23:04:35 2014
@@ -698,7 +698,7 @@ public class PigServer {
             grunt.parseStopOnError(true);
         } catch (org.apache.pig.tools.pigscript.parser.ParseException e) {
             log.error(e.getLocalizedMessage());
-            throw new IOException(e.getCause());
+            throw new IOException(e);
         }
     }
 

Modified: pig/trunk/src/org/apache/pig/tools/ToolsPigServer.java
URL: http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/tools/ToolsPigServer.java?rev=1577753&r1=1577752&r2=1577753&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/tools/ToolsPigServer.java (original)
+++ pig/trunk/src/org/apache/pig/tools/ToolsPigServer.java Fri Mar 14 23:04:35 2014
@@ -104,7 +104,7 @@ public class ToolsPigServer extends PigS
             substituted = pigContext.doParamSubstitution(fis, paramMapToList(params), paramFiles);
         }catch (FileNotFoundException e){
             log.error(e.getLocalizedMessage());
-            throw new IOException(e.getCause());
+            throw new IOException(e);
         } finally {
             if (fis != null) {
                 fis.close();
@@ -120,7 +120,7 @@ public class ToolsPigServer extends PigS
             grunt.parseOnly();
         } catch (org.apache.pig.tools.pigscript.parser.ParseException e) {
             log.error(e.getLocalizedMessage());
-            throw new IOException(e.getCause());
+            throw new IOException(e);
         }
 
         Graph g = getClonedGraph();

Modified: pig/trunk/test/org/apache/pig/pigunit/pig/PigServer.java
URL: http://svn.apache.org/viewvc/pig/trunk/test/org/apache/pig/pigunit/pig/PigServer.java?rev=1577753&r1=1577752&r2=1577753&view=diff
==============================================================================
--- pig/trunk/test/org/apache/pig/pigunit/pig/PigServer.java (original)
+++ pig/trunk/test/org/apache/pig/pigunit/pig/PigServer.java Fri Mar 14 23:04:35 2014
@@ -54,11 +54,9 @@ public class PigServer extends org.apach
       grunt.setInteractive(false);
       grunt.parseStopOnError(true);
     } catch (FileNotFoundException e) {
-      e.printStackTrace();
-      throw new IOException(e.getCause());
+      throw new IOException(e);
     } catch (org.apache.pig.tools.pigscript.parser.ParseException e) {
-      e.printStackTrace();
-      throw new IOException(e.getCause());
+      throw new IOException(e);
     }
   }
 }



Mime
View raw message