----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/69663/ -----------------------------------------------------------
(Updated Jan. 9, 2019, 5:50 p.m.) Review request for hive, Ashutosh Chauhan, Gopal V, Jesús Camacho Rodríguez, and Jason Dere. Changes ------- Updated the patch with review comments. Bugs: HIVE-16976 https://issues.apache.org/jira/browse/HIVE-16976 Repository: hive-git Description ------- DPP: SyntheticJoinPredicate transitivity for < > and BETWEEN The patch supports predicates on non-equi joins and provides an interface for storage handler to decide if it can use this optimization. Work to integrate this with DPP and semijoin will be done in separate JIRA. Diffs (updated) ----- ql/src/java/org/apache/hadoop/hive/ql/exec/FunctionRegistry.java d7f069eaa7 ql/src/java/org/apache/hadoop/hive/ql/metadata/HiveStorageHandler.java 2ebb149354 ql/src/java/org/apache/hadoop/hive/ql/optimizer/DynamicPartitionPruningOptimization.java a1401aac72 ql/src/java/org/apache/hadoop/hive/ql/parse/GenTezUtils.java f8c7e18eb1 ql/src/java/org/apache/hadoop/hive/ql/plan/ExprNodeDynamicListDesc.java 676dfc9421 ql/src/java/org/apache/hadoop/hive/ql/ppd/SyntheticJoinPredicate.java e97e44796f ql/src/test/results/clientpositive/llap/cross_prod_1.q.out f900a01be4 ql/src/test/results/clientpositive/llap/groupby_groupingset_bug.q.out de74af6dff ql/src/test/results/clientpositive/llap/semijoin.q.out 00bc6cec55 ql/src/test/results/clientpositive/llap/subquery_in.q.out 07cc4dbabc ql/src/test/results/clientpositive/llap/subquery_notin.q.out 29d8bbfb48 ql/src/test/results/clientpositive/llap/subquery_scalar.q.out 1cf281afbd ql/src/test/results/clientpositive/llap/subquery_select.q.out 6255abdd70 Diff: https://reviews.apache.org/r/69663/diff/2/ Changes: https://reviews.apache.org/r/69663/diff/1-2/ Testing ------- Thanks, Deepak Jaiswal