You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by ya...@apache.org on 2010/11/22 18:28:17 UTC

svn commit: r1037782 - in /pig/trunk: CHANGES.txt src/org/apache/pig/tools/pigstats/JobStats.java

Author: yanz
Date: Mon Nov 22 17:28:17 2010
New Revision: 1037782

URL: http://svn.apache.org/viewvc?rev=1037782&view=rev
Log:
PIG-1739: zero status is returned when pig script fails (yanz)

Modified:
    pig/trunk/CHANGES.txt
    pig/trunk/src/org/apache/pig/tools/pigstats/JobStats.java

Modified: pig/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/trunk/CHANGES.txt?rev=1037782&r1=1037781&r2=1037782&view=diff
==============================================================================
--- pig/trunk/CHANGES.txt (original)
+++ pig/trunk/CHANGES.txt Mon Nov 22 17:28:17 2010
@@ -220,6 +220,8 @@ PIG-1309: Map-side Cogroup (ashutoshc)
 
 BUG FIXES
 
+PIG-1739: zero status is returned when pig script fails (yanz)
+
 PIG-1738: New logical plan: Optimized UserFuncExpression.getFieldSchema (daijy)
 
 PIG-1732: New logical plan: logical plan get confused if we generate the same

Modified: pig/trunk/src/org/apache/pig/tools/pigstats/JobStats.java
URL: http://svn.apache.org/viewvc/pig/trunk/src/org/apache/pig/tools/pigstats/JobStats.java?rev=1037782&r1=1037781&r2=1037782&view=diff
==============================================================================
--- pig/trunk/src/org/apache/pig/tools/pigstats/JobStats.java (original)
+++ pig/trunk/src/org/apache/pig/tools/pigstats/JobStats.java Mon Nov 22 17:28:17 2010
@@ -431,7 +431,7 @@ public final class JobStats extends Oper
         if (mapStores.size() + reduceStores.size() == 1) {
             POStore sto = (mapStores.size() > 0) ? mapStores.get(0)
                     : reduceStores.get(0);
-            if (!sto.isTmpStore()) {
+            if (!sto.isTmpStore() || state != JobState.SUCCESS) {
                 long records = (mapStores.size() > 0) ? mapOutputRecords
                         : reduceOutputRecords;           
                 OutputStats ds = new OutputStats(sto.getSFile().getFileName(),