----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57903/ -----------------------------------------------------------
(Updated March 24, 2017, 9:20 p.m.) Review request for hive, Gopal V, Jason Dere, and Siddharth Seth. Changes ------- Fixed the hang and added a unit test. Bugs: HIVE-16282 https://issues.apache.org/jira/browse/HIVE-16282 Repository: hive-git Description ------- Currently all reducers have slow start meaning, the mappers get preference in scheduling. However, incase of semijoin reductions, the reducer of semijoin branch should get priority before the mapper to which it feeds in. Diffs (updated) ----- ql/src/java/org/apache/hadoop/hive/ql/exec/tez/DagUtils.java aa2dfc7 ql/src/java/org/apache/hadoop/hive/ql/optimizer/DynamicPartitionPruningOptimization.java b9f5912 ql/src/java/org/apache/hadoop/hive/ql/parse/GenTezUtils.java 905431f ql/src/java/org/apache/hadoop/hive/ql/parse/GenTezWork.java 97f3300 ql/src/java/org/apache/hadoop/hive/ql/plan/ReduceSinkDesc.java 38461d5 ql/src/java/org/apache/hadoop/hive/ql/plan/ReduceWork.java ee784dc ql/src/java/org/apache/hadoop/hive/ql/plan/TezEdgeProperty.java a3aa12f ql/src/test/queries/clientpositive/dynamic_semijoin_reduction.q 6338ac3 ql/src/test/results/clientpositive/llap/dynamic_semijoin_reduction.q.out c1dd8c0 Diff: https://reviews.apache.org/r/57903/diff/2/ Changes: https://reviews.apache.org/r/57903/diff/1-2/ Testing ------- Thanks, Deepak Jaiswal