> On April 28, 2014, 5:22 a.m., Daniel Dai wrote: > > src/org/apache/pig/tools/pigstats/PigStatusReporter.java, line 76 > > <https://reviews.apache.org/r/20761/diff/1/?file=569302#file569302line76> > > > > While incrCounter is useful but seems this is a separate refactory, am > > I right? > > Cheolsoo Park wrote: > No, it's actually required. > > Hadoop counter is required for MR, whereas Tez counter is required for > Tez. Since I cannot overload getCounter() for different return type, I wrap > it with incrCounter().
I see, so if UDF relies on PigStatusReporter.getCounter() will not be able to propagate counters in Tez. Can we put a comment on getCounter/incrCounter to clarify it? - Daniel ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20761/#review41579 ----------------------------------------------------------- On April 27, 2014, 9:02 p.m., Cheolsoo Park wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20761/ > ----------------------------------------------------------- > > (Updated April 27, 2014, 9:02 p.m.) > > > Review request for pig, Daniel Dai and Rohini Palaniswamy. > > > Bugs: PIG-3860 > https://issues.apache.org/jira/browse/PIG-3860 > > > Repository: pig-git > > > Description > ------- > > The patch includes the following changes- > > 1) Introduce a generic class called TaskContext<?>. > - For MR, it holds TaskInputOutputContext > - For Tez, it holds TezProcessorContext > - For Fetch, it holds FetchContext > Note that it is under the shims package because TezProcessorContext is not > visible when -Dhadoopversion=20. > 2) Refactor PigStatusReporter and PigHadoopLogger by using generic > TaskContext instead of MR-specific TaskInputOutputContext. > 3) Whitespace fixes. > > The diff file in this RB request is for tez branch. I'll post another to the > jira for trunk. > > > Diffs > ----- > > > shims/src/hadoop20/org/apache/pig/backend/hadoop/executionengine/shims/TaskContext.java > e69de29 > > shims/src/hadoop23/org/apache/pig/backend/hadoop/executionengine/shims/TaskContext.java > e69de29 > src/org/apache/pig/TypedOutputEvalFunc.java 5c73bf5 > src/org/apache/pig/backend/hadoop/executionengine/fetch/FetchLauncher.java > 689e424 > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/MapReducePOStoreImpl.java > 215ff89 > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigCombiner.java > b41648f > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigGenericMapBase.java > c52603f > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigGenericMapReduce.java > bb10f0c > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigHadoopLogger.java > 4db4a11 > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigMapReduceCounter.java > 2695be3 > > src/org/apache/pig/backend/hadoop/executionengine/mapReduceLayer/PigRecordReader.java > 0c221e0 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/PigLogger.java > 4ef44e8 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/expressionOperators/POUserFunc.java > 16b6fb9 > > src/org/apache/pig/backend/hadoop/executionengine/physicalLayer/util/MonitoredUDFExecutor.java > 8fc0c65 > src/org/apache/pig/backend/hadoop/executionengine/tez/PigProcessor.java > 7c8286a > src/org/apache/pig/tools/counters/PigCounterHelper.java 0023035 > src/org/apache/pig/tools/pigstats/PigStatusReporter.java 5b1bc42 > > Diff: https://reviews.apache.org/r/20761/diff/ > > > Testing > ------- > > All unit tests pass. > All e2e tests pass except Warning_4 (known - PIG-3739). > > > Thanks, > > Cheolsoo Park > >
