-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/20873/
-----------------------------------------------------------
Review request for Tajo.
Bugs: TAJO-801
https://issues.apache.org/jira/browse/TAJO-801
Repository: tajo
Description
-------
Currently the following query is not supported.
{code}
default> select id, count(distinct age), count(distinct name) from table2 group
by id;
ERROR: different DISTINCT columns are not supported yet: age, name
{code}
Diffs
-----
tajo-common/src/main/java/org/apache/tajo/util/TUtil.java e116000
tajo-core/src/main/java/org/apache/tajo/engine/eval/EvalTreeUtil.java 38c7e1f
tajo-core/src/main/java/org/apache/tajo/engine/parser/HiveQLAnalyzer.java
60e9685
tajo-core/src/main/java/org/apache/tajo/engine/planner/BasicLogicalPlanVisitor.java
0f758bf
tajo-core/src/main/java/org/apache/tajo/engine/planner/ExplainLogicalPlanVisitor.java
9dd8700
tajo-core/src/main/java/org/apache/tajo/engine/planner/ExprAnnotator.java
be0259b
tajo-core/src/main/java/org/apache/tajo/engine/planner/LogicalPlanVisitor.java
17b5d0a
tajo-core/src/main/java/org/apache/tajo/engine/planner/PhysicalPlannerImpl.java
2053e36
tajo-core/src/main/java/org/apache/tajo/engine/planner/PlannerUtil.java
9f988bd
tajo-core/src/main/java/org/apache/tajo/engine/planner/PreLogicalPlanVerifier.java
1ee0878
tajo-core/src/main/java/org/apache/tajo/engine/planner/enforce/Enforcer.java
91190f6
tajo-core/src/main/java/org/apache/tajo/engine/planner/global/GlobalPlanner.java
edc08fc
tajo-core/src/main/java/org/apache/tajo/engine/planner/global/builder/DistinctGroupbyBuilder.java
PRE-CREATION
tajo-core/src/main/java/org/apache/tajo/engine/planner/logical/DistinctGroupbyNode.java
PRE-CREATION
tajo-core/src/main/java/org/apache/tajo/engine/planner/logical/GroupbyNode.java
bafe0c6
tajo-core/src/main/java/org/apache/tajo/engine/planner/logical/NodeType.java
f498231
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/DistinctGroupbyHashAggregationExec.java
PRE-CREATION
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/DistinctGroupbySortAggregationExec.java
PRE-CREATION
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/ExternalSortExec.java
c422b49
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/HashAggregateExec.java
a31ad90
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/SeqScanExec.java
6dbcc3f
tajo-core/src/main/java/org/apache/tajo/engine/planner/physical/SortAggregateExec.java
629889d
tajo-core/src/main/proto/TajoWorkerProtocol.proto 5d4ae44
tajo-core/src/test/java/org/apache/tajo/engine/query/TestGroupByQuery.java
9e3c375
tajo-core/src/test/resources/queries/TestGroupByQuery/testDistinctAggregation7.sql
PRE-CREATION
tajo-core/src/test/resources/queries/TestGroupByQuery/testDistinctAggregation_case1.sql
PRE-CREATION
tajo-core/src/test/resources/queries/TestGroupByQuery/testDistinctAggregation_case2.sql
PRE-CREATION
tajo-core/src/test/resources/queries/TestGroupByQuery/testDistinctAggregation_case3.sql
PRE-CREATION
tajo-core/src/test/resources/queries/TestGroupByQuery/testDistinctAggregation_case4.sql
PRE-CREATION
tajo-core/src/test/resources/queries/TestGroupByQuery/testDistinctAggregation_case5.sql
PRE-CREATION
tajo-core/src/test/resources/queries/TestGroupByQuery/testDistinctAggregation_case6.sql
PRE-CREATION
tajo-core/src/test/resources/queries/TestGroupByQuery/testDistinctAggregation_case7.sql
PRE-CREATION
tajo-core/src/test/resources/results/TestGroupByQuery/testDistinctAggregation7.result
PRE-CREATION
tajo-core/src/test/resources/results/TestGroupByQuery/testDistinctAggregation_case1.result
PRE-CREATION
tajo-core/src/test/resources/results/TestGroupByQuery/testDistinctAggregation_case2.result
PRE-CREATION
tajo-core/src/test/resources/results/TestGroupByQuery/testDistinctAggregation_case3.result
PRE-CREATION
tajo-core/src/test/resources/results/TestGroupByQuery/testDistinctAggregation_case4.result
PRE-CREATION
tajo-core/src/test/resources/results/TestGroupByQuery/testDistinctAggregation_case5.result
PRE-CREATION
tajo-core/src/test/resources/results/TestGroupByQuery/testDistinctAggregation_case6.result
PRE-CREATION
tajo-core/src/test/resources/results/TestGroupByQuery/testDistinctAggregation_case7.result
PRE-CREATION
Diff: https://reviews.apache.org/r/20873/diff/
Testing
-------
mvn clean install;
Tested in local cluster with tpch 100 scale.
Thanks,
hyoungjun kim
