-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35163/
-----------------------------------------------------------
Review request for drill, Aman Sinha and Jinfeng Ni.
Bugs: DRILL-1169
https://issues.apache.org/jira/browse/DRILL-1169
Repository: drill-git
Description
-------
Use Union-All followed by Aggregate to implement Union-Distinct
Diffs
-----
exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillDistinctUnionByUnionAll.java
PRE-CREATION
exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillRuleSets.java
655ad92
exec/java-exec/src/main/java/org/apache/drill/exec/planner/logical/DrillUnionRel.java
566c558
exec/java-exec/src/main/java/org/apache/drill/exec/planner/sql/parser/UnsupportedOperatorsVisitor.java
f1ec851
exec/java-exec/src/test/java/org/apache/drill/TestDisabledFunctionality.java
adbf653
exec/java-exec/src/test/java/org/apache/drill/TestUnionDistinct.java
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q1.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q10.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q11.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q12.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q13.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q14.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q15.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q16.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q17.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q18_1.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q18_2.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q18_3.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q2.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q3.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q4.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q5.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q6.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q6_1.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q7.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q8.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/q9.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/testAggregationOnUnionDistinctOperator_1.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/testAggregationOnUnionDistinctOperator_2.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/testProjectDownOverUnionDistinctImplicitCasting.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/testProjectFiltertPushDownOverUnionDistinct.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/testProjectPushDownOverUnionDistinctWithProject.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/testProjectPushDownOverUnionDistinctWithoutProject.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/testProjectPushDownProjectColumnReorderingAndAlias.tsv
PRE-CREATION
exec/java-exec/src/test/resources/testframework/TestUnionDistinctQueries/testProjectWithExpressionPushDownOverUnionDistinct.tsv
PRE-CREATION
Diff: https://reviews.apache.org/r/35163/diff/
Testing
-------
all the required tests
Thanks,
Sean Hsuan-Yi Chu
