----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25280/ -----------------------------------------------------------
(Updated Sept. 5, 2014, 6:20 p.m.) Review request for hive, Brock Noland and Xuefu Zhang. Bugs: HIVE-7939 https://issues.apache.org/jira/browse/HIVE-7939 Repository: hive-git Description ------- Currently, GraphTran uses its own execute method, which executes the operator plan in a DFS fashion, and does something special for union. The goal for this JIRA is to do some refactoring and make it conform to the SparkTran interface. The initial idea is to use varargs for SparkTran::transform. Diffs ----- ql/src/java/org/apache/hadoop/hive/ql/exec/spark/GraphTran.java 5d4414a ql/src/java/org/apache/hadoop/hive/ql/exec/spark/MapTran.java b03a51c ql/src/java/org/apache/hadoop/hive/ql/exec/spark/ReduceTran.java 76b74e7 ql/src/java/org/apache/hadoop/hive/ql/exec/spark/SparkPlan.java 46e4b6d ql/src/java/org/apache/hadoop/hive/ql/exec/spark/SparkPlanGenerator.java 9b11fe4 ql/src/java/org/apache/hadoop/hive/ql/exec/spark/SparkTran.java 19894b0 ql/src/java/org/apache/hadoop/hive/ql/exec/spark/UnionTran.java 5ec7d0f ql/src/test/results/clientpositive/spark/union17.q.out.sorted PRE-CREATION ql/src/test/results/clientpositive/spark/union20.q.out.sorted PRE-CREATION ql/src/test/results/clientpositive/spark/union21.q.out.sorted PRE-CREATION ql/src/test/results/clientpositive/spark/union27.q.out.sorted PRE-CREATION Diff: https://reviews.apache.org/r/25280/diff/ Testing ------- Thanks, Chao Sun