-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/15390/
-----------------------------------------------------------
(Updated Nov. 11, 2013, 6:55 p.m.)
Review request for pig.
Changes
-------
I ran full unit tests and fixed TestPigRunner.testRegisterExternalJar.
With the updated patch, all unit tests pass now.
Bugs: PIG-3561
https://issues.apache.org/jira/browse/PIG-3561
Repository: pig-git
Description
-------
The attached patch factors out methods/fields that can be used by MR and non-MR
backends into PigStats and JobStats. PIG-3419 moved them to SimplePigStats and
MRJobStats, but I found they are not MR-specific and applicable to non-MR
backends such as Tez backend.
Diffs (updated)
-----
src/org/apache/pig/tools/pigstats/JobStats.java 5eac24b
src/org/apache/pig/tools/pigstats/PigStats.java 6d2e58e
src/org/apache/pig/tools/pigstats/mapreduce/MRJobStats.java 1a37848
src/org/apache/pig/tools/pigstats/mapreduce/MRPigStatsUtil.java 4bdcf19
src/org/apache/pig/tools/pigstats/mapreduce/SimplePigStats.java bbfd5a9
test/org/apache/pig/test/TestPigRunner.java c5d84cf
Diff: https://reviews.apache.org/r/15390/diff/
Testing
-------
ant test-commit
Thanks,
Cheolsoo Park