You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pig.apache.org by an...@apache.org on 2014/05/31 19:10:37 UTC
svn commit: r1598889 - in /pig/branches/branch-0.13: CHANGES.txt
src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigHadoopLogger.java
Author: aniket486
Date: Sat May 31 17:10:36 2014
New Revision: 1598889
URL: http://svn.apache.org/r1598889
Log:
PIG-3739: The Warning_4 e2e test is broken in trunk (aniket486)
Modified:
pig/branches/branch-0.13/CHANGES.txt
pig/branches/branch-0.13/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigHadoopLogger.java
Modified: pig/branches/branch-0.13/CHANGES.txt
URL: http://svn.apache.org/viewvc/pig/branches/branch-0.13/CHANGES.txt?rev=1598889&r1=1598888&r2=1598889&view=diff
==============================================================================
--- pig/branches/branch-0.13/CHANGES.txt (original)
+++ pig/branches/branch-0.13/CHANGES.txt Sat May 31 17:10:36 2014
@@ -147,6 +147,8 @@ PIG-3882: Multiquery off mode execution
BUG FIXES
+PIG-3739: The Warning_4 e2e test is broken in trunk (aniket486)
+
PIG-3976: Typo correction in JobStats breaks Oozie (rohini)
PIG-3874: FileLocalizer temp path can sometimes be non-unique (chitnis via cheolsoo)
Modified: pig/branches/branch-0.13/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigHadoopLogger.java
URL: http://svn.apache.org/viewvc/pig/branches/branch-0.13/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigHadoopLogger.java?rev=1598889&r1=1598888&r2=1598889&view=diff
==============================================================================
--- pig/branches/branch-0.13/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigHadoopLogger.java (original)
+++ pig/branches/branch-0.13/src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigHadoopLogger.java Sat May 31 17:10:36 2014
@@ -81,9 +81,9 @@ public final class PigHadoopLogger imple
}
if (o instanceof EvalFunc || o instanceof LoadFunc || o instanceof StoreFunc) {
reporter.incrCounter(className, warningEnum.name(), 1);
- } else {
- reporter.incrCounter(warningEnum, 1);
}
+ // For backwards compatibility, always report with warningEnum, see PIG-3739
+ reporter.incrCounter(warningEnum, 1);
} else {
//TODO:
//in local mode of execution if the PigHadoopLogger is used initially,