You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@asterixdb.apache.org by AsterixDB Code Review <do...@asterix-gerrit.ics.uci.edu> on 2025/02/12 07:04:10 UTC
Change in asterixdb[master]: [ASTERIXDB-3434][RT]: Fixed Group By Memmory Issue
From Gaurav Vaghasiya <gv...@gmail.com>:
Attention is currently required from: Ali Alsuliman, Gaurav Vaghasiya.
Hello Ian Maxon, Ali Alsuliman, Jenkins, Anon. E. Moose #1000171,
I'd like you to reexamine a change. Please visit
https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18392
to look at the new patch set (#8).
Change subject: [ASTERIXDB-3434][RT]: Fixed Group By Memmory Issue
......................................................................
[ASTERIXDB-3434][RT]: Fixed Group By Memmory Issue
Details:
Each Group By Operator(Sort and Hash) will use half memory of total Group memory to accomdate both local and global Group By.
Updated cc.conf to accommodate new changes.
Change-Id: I3973791350ea74bb695937f4937f60cc65592dd5
---
M asterixdb/asterix-app/src/main/resources/cc3.conf
M asterixdb/asterix-app/src/test/resources/cc-cloud-storage-azblob.conf
M asterixdb/asterix-app/src/test/resources/cc-cbotest.conf
M hyracks-fullstack/algebricks/algebricks-rewriter/src/main/java/org/apache/hyracks/algebricks/rewriter/rules/SetMemoryRequirementsRule.java
M asterixdb/asterix-app/src/test/resources/cc-cloud-storage.conf
M asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate/min_max_arrays/min_max_arrays.35.query.sqlpp
M asterixdb/asterix-app/src/test/resources/cc-ssl.conf
M asterixdb/asterix-app/src/test/resources/cc-main.conf
M asterixdb/asterix-app/src/test/resources/runtimets/results/api/cluster_state_1_less/cluster_state_1_less.1.regexadm
M asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/aggregate-sql/min_max_arrays/min_max_arrays.32.query.sqlpp
M asterixdb/asterix-app/src/test/resources/runtimets/queries_sqlpp/misc/metadata_only_01/metadata_only_01.2.plans.sqlpp
M asterixdb/asterix-app/src/main/resources/cc4.conf
M asterixdb/asterix-app/src/test/resources/cc-single.conf
M asterixdb/asterix-app/src/test/resources/cc-small-txn-log-partition.conf
M asterixdb/asterix-app/src/test/resources/cc-single-partition.conf
M asterixdb/asterix-app/src/test/resources/cc-columnar.conf
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/cluster_state_1.1.regexadm
M asterixdb/asterix-app/src/main/resources/cc_static_partitioning.conf
M asterixdb/asterix-app/src/test/resources/cc-storage.conf
M asterixdb/asterix-app/src/test/resources/cc-cloud-storage-gcs.conf
M asterixdb/asterix-app/src/test/resources/cc-cloud-storage-main.conf
M asterixdb/asterix-app/src/test/resources/cc-single-lookup.conf
M asterixdb/asterix-app/src/main/resources/cc2.conf
M asterixdb/asterix-app/src/test/resources/cc.conf
M asterixdb/asterix-app/src/main/resources/cc.conf
M asterixdb/asterix-app/src/main/resources/cc_no_cbo.conf
M asterixdb/asterix-app/src/test/resources/cc-multipart.conf
28 files changed, 59 insertions(+), 28 deletions(-)
git pull ssh://asterix-gerrit.ics.uci.edu:29418/asterixdb refs/changes/92/18392/8
--
To view, visit https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/18392
To unsubscribe, or for help writing mail filters, visit https://asterix-gerrit.ics.uci.edu/settings
Gerrit-Project: asterixdb
Gerrit-Branch: master
Gerrit-Change-Id: I3973791350ea74bb695937f4937f60cc65592dd5
Gerrit-Change-Number: 18392
Gerrit-PatchSet: 8
Gerrit-Owner: Gaurav Vaghasiya <gv...@gmail.com>
Gerrit-Reviewer: Ali Alsuliman <al...@gmail.com>
Gerrit-Reviewer: Anon. E. Moose #1000171
Gerrit-Reviewer: Ian Maxon <im...@apache.org>
Gerrit-Reviewer: Jenkins <je...@fulliautomatix.ics.uci.edu>
Gerrit-CC: Murtadha Hubail <mh...@apache.org>
Gerrit-Attention: Ali Alsuliman <al...@gmail.com>
Gerrit-Attention: Gaurav Vaghasiya <gv...@gmail.com>
Gerrit-MessageType: newpatchset