diff options
| author | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2020-11-20 14:41:14 +0200 |
|---|---|---|
| committer | Heikki Linnakangas <heikki.linnakangas@iki.fi> | 2020-11-20 14:41:14 +0200 |
| commit | 340ae3cfb8f38a49664c0da3ff033346e92f0450 (patch) | |
| tree | 073d79a69488e6f74c1aeca31d4475197b2e7caf | |
| parent | 9e9a31bd009663bcdd5e676d6012ee691d8944b6 (diff) | |
| download | postgresql-340ae3cfb8f38a49664c0da3ff033346e92f0450.tar.gz postgresql-340ae3cfb8f38a49664c0da3ff033346e92f0450.zip | |
Skip allocating hash table in EXPLAIN-only mode.
This is a backpatch of commit 2cccb627f1, backpatched due to popular
demand. Backpatch to all supported versions.
Author: Alexey Bashtanov
Discussion: https://www.postgresql.org/message-id/36823f65-050d-ae24-aa4d-a37726998240%40imap.cc
| -rw-r--r-- | src/backend/executor/nodeAgg.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/src/backend/executor/nodeAgg.c b/src/backend/executor/nodeAgg.c index 75e5bbf209d..8a0bd887bf0 100644 --- a/src/backend/executor/nodeAgg.c +++ b/src/backend/executor/nodeAgg.c @@ -3665,7 +3665,11 @@ ExecInitAgg(Agg *node, EState *estate, int eflags) &aggstate->hash_ngroups_limit, &aggstate->hash_planned_partitions); find_hash_columns(aggstate); - build_hash_tables(aggstate); + + /* Skip massive memory allocation if we are just doing EXPLAIN */ + if (!(eflags & EXEC_FLAG_EXPLAIN_ONLY)) + build_hash_tables(aggstate); + aggstate->table_filled = false; /* Initialize this to 1, meaning nothing spilled, yet */ |