----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/27283/ -----------------------------------------------------------
Review request for hive, Szehon Ho and Xuefu Zhang. Bugs: HIVE-8610 https://issues.apache.org/jira/browse/HIVE-8610 Repository: hive-git Description ------- This patch adds QBJoinTree and colExprMap for the cloned join operator tree in SkewJoinOptimizer, so that CommonJoinResolver can properly convert the cloned join to map join. The added tests are copied from skewjoinopt*.q, except that auto map join is enabled. Diffs ----- ql/src/java/org/apache/hadoop/hive/ql/optimizer/SkewJoinOptimizer.java e87c41b ql/src/test/queries/clientpositive/skewjoin_mapjoin1.q PRE-CREATION ql/src/test/queries/clientpositive/skewjoin_mapjoin10.q PRE-CREATION ql/src/test/queries/clientpositive/skewjoin_mapjoin11.q PRE-CREATION ql/src/test/queries/clientpositive/skewjoin_mapjoin2.q PRE-CREATION ql/src/test/queries/clientpositive/skewjoin_mapjoin3.q PRE-CREATION ql/src/test/queries/clientpositive/skewjoin_mapjoin4.q PRE-CREATION ql/src/test/queries/clientpositive/skewjoin_mapjoin5.q PRE-CREATION ql/src/test/queries/clientpositive/skewjoin_mapjoin6.q PRE-CREATION ql/src/test/queries/clientpositive/skewjoin_mapjoin7.q PRE-CREATION ql/src/test/queries/clientpositive/skewjoin_mapjoin8.q PRE-CREATION ql/src/test/queries/clientpositive/skewjoin_mapjoin9.q PRE-CREATION ql/src/test/results/clientpositive/skewjoin_mapjoin1.q.out PRE-CREATION ql/src/test/results/clientpositive/skewjoin_mapjoin10.q.out PRE-CREATION ql/src/test/results/clientpositive/skewjoin_mapjoin11.q.out PRE-CREATION ql/src/test/results/clientpositive/skewjoin_mapjoin2.q.out PRE-CREATION ql/src/test/results/clientpositive/skewjoin_mapjoin3.q.out PRE-CREATION ql/src/test/results/clientpositive/skewjoin_mapjoin4.q.out PRE-CREATION ql/src/test/results/clientpositive/skewjoin_mapjoin5.q.out PRE-CREATION ql/src/test/results/clientpositive/skewjoin_mapjoin6.q.out PRE-CREATION ql/src/test/results/clientpositive/skewjoin_mapjoin7.q.out PRE-CREATION ql/src/test/results/clientpositive/skewjoin_mapjoin8.q.out PRE-CREATION ql/src/test/results/clientpositive/skewjoin_mapjoin9.q.out PRE-CREATION Diff: https://reviews.apache.org/r/27283/diff/ Testing ------- Thanks, Rui Li