----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27640/ -----------------------------------------------------------
(Updated Nov. 5, 2014, 8:29 p.m.) Review request for hive, Chao Sun, Jimmy Xiang, Szehon Ho, and Xuefu Zhang. Summary (updated) ----------------- HIVE-8700 Replace ReduceSink to HashTableSink (or equi.) for small tables [Spark Branch] 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 ----- ql/src/java/org/apache/hadoop/hive/ql/optimizer/physical/SparkMapJoinResolver.java PRE-CREATION ql/src/java/org/apache/hadoop/hive/ql/parse/spark/SparkCompiler.java 795a5d7 Diff: https://reviews.apache.org/r/27640/diff/ Testing ------- Thanks, Suhas Satish
