>From Shahrzad Shirazi <[email protected]>:
Attention is currently required from: Ali Alsuliman, Shahrzad Shirazi.
Hello Ali Alsuliman, Anon. E. Moose #1000171, Jenkins,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21292?usp=email
to look at the new patch set (#19).
The following approvals got outdated and were removed:
Code-Review+1 by Shahrzad Shirazi, Contrib+1 by Anon. E. Moose #1000171,
Integration-Tests+1 by Jenkins, Verified+1 by Jenkins
Change subject: [ASTERIXDB-3783][COMP] Implement hash-based COUNT(DISTINCT)
aggregation
......................................................................
[ASTERIXDB-3783][COMP] Implement hash-based COUNT(DISTINCT) aggregation
- user model changes: no
- storage format changes: no
- interface changes: no
Ext-ref: MB-72573
Change-Id: Ia6e37080a581292111ddc1366b885526412c16ac
---
M
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/compiler/provider/SqlppCompilationProvider.java
M
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/base/RuleCollections.java
A
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/RewriteCountDistinctToHashRule.java
M
asterixdb/asterix-algebra/src/main/java/org/apache/asterix/optimizer/rules/RewriteDistinctAggregateRule.java
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/distinct/hashbased/hashbased.1.ddl.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/distinct/hashbased/hashbased.10.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/distinct/hashbased/hashbased.2.update.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/distinct/hashbased/hashbased.3.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/distinct/hashbased/hashbased.4.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/distinct/hashbased/hashbased.5.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/distinct/hashbased/hashbased.6.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/distinct/hashbased/hashbased.7.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/distinct/hashbased/hashbased.8.query.sqlpp
A
asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/distinct/hashbased/hashbased.9.query.sqlpp
M
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1/cluster_state_1.1.regexadm
M
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1_full/cluster_state_1_full.1.regexadm
M
asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1_less/cluster_state_1_less.1.regexadm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/distinct/hashbased/hasbased.3.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/distinct/hashbased/hasbased.4.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/distinct/hashbased/hasbased.5.plan
A
asterixdb/asterix-app/src/test/resources/runtimets/results/distinct/hashbased/hasbased.6.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/distinct/hashbased/hasbased.7.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/distinct/hashbased/hashbased.10.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/distinct/hashbased/hashbased.8.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results/distinct/hashbased/hashbased.9.adm
A
asterixdb/asterix-app/src/test/resources/runtimets/results_column/distinct/hashbased/hasbased.5.plan
M asterixdb/asterix-app/src/test/resources/runtimets/sqlpp_queries.xml
M
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/CompilerProperties.java
M
asterixdb/asterix-common/src/main/java/org/apache/asterix/common/config/OptimizationConfUtil.java
M
asterixdb/asterix-om/src/main/java/org/apache/asterix/om/functions/BuiltinFunctions.java
A
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/aggregates/std/HashDistinctCountAggregateDescriptor.java
A
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/aggregates/std/HashDistinctCountAggregateFunction.java
M
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/functions/FunctionCollection.java
M
asterixdb/asterix-runtime/src/main/java/org/apache/asterix/runtime/functions/FunctionTypeInferers.java
M
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/config/AlgebricksConfig.java
M
hyracks-fullstack/algebricks/algebricks-core/src/main/java/org/apache/hyracks/algebricks/core/rewriter/base/PhysicalOptimizationConfig.java
36 files changed, 1,103 insertions(+), 2 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb
refs/changes/92/21292/19
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/21292?usp=email
To unsubscribe, or for help writing mail filters, visit
https://asterix-gerrit.ics.uci.edu/settings?usp=email
Gerrit-MessageType: newpatchset
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: Ia6e37080a581292111ddc1366b885526412c16ac
Gerrit-Change-Number: 21292
Gerrit-PatchSet: 19
Gerrit-Owner: Shahrzad Shirazi <[email protected]>
Gerrit-Reviewer: Ali Alsuliman <[email protected]>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Jenkins <[email protected]>
Gerrit-Reviewer: Shahrzad Shirazi <[email protected]>
Gerrit-Attention: Shahrzad Shirazi <[email protected]>
Gerrit-Attention: Ali Alsuliman <[email protected]>
