-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56043/
-----------------------------------------------------------

Review request for pig and Daniel Dai.


Bugs: PIG-5118
    https://issues.apache.org/jira/browse/PIG-5118


Repository: pig


Description
-------

The current code of splitting DAG does not handle cases where there is split 
and a clean segmentation is not possible. It also does not handle cases where 
there is multiple segments and they are later used together. Both cases it ends 
up with "Invalid dag containing 0 vertices" error as it creates one empty DAG 
and and a new DAG which contains the full original plan as it was not able to 
segment.


Diffs
-----

  
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/tez/TezLauncher.java
 1780431 
  
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/tez/plan/TezCompiler.java
 1780431 
  
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/tez/plan/TezOperPlan.java
 1780431 
  
http://svn.apache.org/repos/asf/pig/trunk/src/org/apache/pig/backend/hadoop/executionengine/tez/plan/TezPlanContainer.java
 1780431 
  http://svn.apache.org/repos/asf/pig/trunk/test/e2e/pig/tests/nightly.conf 
1780431 
  
http://svn.apache.org/repos/asf/pig/trunk/test/org/apache/pig/test/data/GoldenFiles/tez/TEZC-LoadStore-2-JDK7.gld
 1780431 
  
http://svn.apache.org/repos/asf/pig/trunk/test/org/apache/pig/test/data/GoldenFiles/tez/TEZC-LoadStore-2.gld
 1780431 
  
http://svn.apache.org/repos/asf/pig/trunk/test/org/apache/pig/tez/TestTezCompiler.java
 1780431 

Diff: https://reviews.apache.org/r/56043/diff/


Testing
-------

e2e tests added


Thanks,

Rohini Palaniswamy

Reply via email to