----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27640/ -----------------------------------------------------------
(Updated Nov. 6, 2014, 11:43 p.m.) Review request for hive, Chao Sun, Jimmy Xiang, Szehon Ho, and Xuefu Zhang. Repository: hive-git Description ------- This replaces ReduceSinks with HashTableSinks in smaller tables for a map-join. But the condition check field to detect map-join is actually being set in CommonJoinResolver, which doesnt exist yet. We need to decide where is the right place to populate this field. Diffs (updated) ----- ql/src/java/org/apache/hadoop/hive/ql/exec/OperatorFactory.java f1c3564 ql/src/java/org/apache/hadoop/hive/ql/optimizer/spark/SparkReduceSinkMapJoinProc.java 92600be Diff: https://reviews.apache.org/r/27640/diff/ Testing ------- Thanks, Suhas Satish