-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/21465/
-----------------------------------------------------------
(Updated May 16, 2014, 3:58 p.m.)
Review request for pig, Daniel Dai and Rohini Palaniswamy.
Changes
-------
Incorporate Rohini's comments.
Bugs: PIG-3918
https://issues.apache.org/jira/browse/PIG-3918
Repository: pig-git
Description
-------
This patch makes PPNL work with Pig-on-Tez:
* Emit event notifications to PPNL in TezLauncher.
* Add a TezJob handle to TezStats. PPNL needs to access to DAG status to report
progress.
* Along the way, two bug fixes-
* TezDagBuilder wasn't setting POLoad#lFile while constructing
POSimpleTezLoad. This causes NPE in PPNL.
* BaseOperatorPlan#connect() shouldn't add nulls to edge maps. This causes
NPE in PPNL.
* Fix unit tests. Some tests were failing after my changes because of
ClassCastException from TezScriptState to MRScriptState.
Diffs (updated)
-----
src/org/apache/pig/backend/hadoop/executionengine/tez/POSimpleTezLoad.java
f5ea1a5
src/org/apache/pig/backend/hadoop/executionengine/tez/TezDagBuilder.java
692cf08
src/org/apache/pig/backend/hadoop/executionengine/tez/TezJob.java da587f8
src/org/apache/pig/backend/hadoop/executionengine/tez/TezJobControl.java
de01a9f
src/org/apache/pig/backend/hadoop/executionengine/tez/TezLauncher.java
f5a8461
src/org/apache/pig/newplan/BaseOperatorPlan.java d86b60b
src/org/apache/pig/tools/pigstats/tez/TezStats.java f94b954
test/org/apache/pig/test/TestAccumulator.java 16f067a
test/org/apache/pig/test/TestBuiltin.java d33ed89
test/org/apache/pig/test/TestLocalRearrange.java 7ffc872
test/org/apache/pig/test/TestSkewedJoin.java 1ce3350
Diff: https://reviews.apache.org/r/21465/diff/
Testing
-------
Thanks,
Cheolsoo Park
