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:57:51 +0200 |
| commit | ded7db4c1dddcead3184b17619ddc5c203db42fe (patch) | |
| tree | 6810f1b3a8f5d68defd8976075f24a910042f808 /src/backend/executor/nodeAgg.c | |
| parent | 2bb8c0f1d94dd4b2d23ae56b2f5ebec785d2df7b (diff) | |
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
Diffstat (limited to 'src/backend/executor/nodeAgg.c')
| -rw-r--r-- | src/backend/executor/nodeAgg.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/backend/executor/nodeAgg.c b/src/backend/executor/nodeAgg.c index a9e1fbb842e..41bfbb75d6f 100644 --- a/src/backend/executor/nodeAgg.c +++ b/src/backend/executor/nodeAgg.c @@ -2254,7 +2254,10 @@ ExecInitAgg(Agg *node, EState *estate, int eflags) if (node->aggstrategy == AGG_HASHED) { - build_hash_table(aggstate); + /* Skip massive memory allocation if we are just doing EXPLAIN */ + if (!(eflags & EXEC_FLAG_EXPLAIN_ONLY)) + build_hash_table(aggstate); + aggstate->table_filled = false; /* Compute the columns we actually need to hash on */ aggstate->hash_needed = find_hash_columns(aggstate); |