----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27933/ -----------------------------------------------------------
(Updated Nov. 13, 2014, 7:17 p.m.) Review request for hive, Chao Sun, Szehon Ho, and Xuefu Zhang. Changes ------- Reverted some changes to HashTableSinkOperator in HIVE-8621. Changed patch v2 so that we can reuse some code. Now SparkHashtableSinkOperator uses HashTableSinkOperator and SparkHashTableSinkDesc extends HashTableSinkDesc. Bugs: HIVE-8810 https://issues.apache.org/jira/browse/HIVE-8810 Repository: hive-git Description ------- Fixed the Spark HashTableSinkOperator Diffs (updated) ----- ql/src/java/org/apache/hadoop/hive/ql/exec/HashTableSinkOperator.java 78d9012 ql/src/java/org/apache/hadoop/hive/ql/exec/OperatorFactory.java f1c3564 ql/src/java/org/apache/hadoop/hive/ql/exec/SparkHashTableSinkOperator.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/exec/spark/HashTableLoader.java 95dad90 ql/src/java/org/apache/hadoop/hive/ql/optimizer/spark/SparkReduceSinkMapJoinProc.java a58a6c5 ql/src/java/org/apache/hadoop/hive/ql/plan/MapredLocalWork.java d566087 ql/src/java/org/apache/hadoop/hive/ql/plan/SparkHashTableSinkDesc.java PRE-CREATION Diff: https://reviews.apache.org/r/27933/diff/ Testing ------- Thanks, Jimmy Xiang