-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/68868/
-----------------------------------------------------------
(Updated Oct. 7, 2018, 4:14 a.m.)
Review request for hive and Jesús Camacho Rodríguez.
Changes
-------
Addressed review comments
Bugs: HIVE-17043
https://issues.apache.org/jira/browse/HIVE-17043
Repository: hive-git
Description
-------
This patch implements/test the following optimizations
* Removal of group by on primary keys
* Reduction of group by keys on primary keys
* is NOT NULL filter removal if NOT NULL constraint is defined
Diffs (updated)
-----
itests/src/test/resources/testconfiguration.properties 9984ce5eed
ql/src/java/org/apache/hadoop/hive/ql/io/orc/VectorizedOrcAcidRowBatchReader.java
66280b2da1
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/HiveDefaultRelMetadataProvider.java
635d27e723
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/RelOptHiveTable.java
42e60de6a8
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveAggregateJoinTransposeRule.java
f43ef01293
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/rules/HiveRelFieldTrimmer.java
5857f730a8
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/stats/EstimateUniqueKeys.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/stats/HiveRelMdColumnUniqueness.java
PRE-CREATION
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/stats/HiveRelMdRowCount.java
1ca1937ed9
ql/src/java/org/apache/hadoop/hive/ql/optimizer/calcite/stats/HiveRelMdUniqueKeys.java
3bf62c535c
ql/src/test/org/apache/hadoop/hive/ql/io/orc/TestVectorizedOrcAcidRowBatchReader.java
0a499b1a1b
ql/src/test/queries/clientpositive/constraints_optimization.q PRE-CREATION
ql/src/test/results/clientpositive/llap/constraints_optimization.q.out
PRE-CREATION
ql/src/test/results/clientpositive/llap/materialized_view_rewrite_1.q.out
735a4db2ee
Diff: https://reviews.apache.org/r/68868/diff/6/
Changes: https://reviews.apache.org/r/68868/diff/5-6/
Testing
-------
Thanks,
Vineet Garg
