diff options
| author | Jeff Davis <jdavis@postgresql.org> | 2020-07-26 14:55:52 -0700 |
|---|---|---|
| committer | Jeff Davis <jdavis@postgresql.org> | 2020-07-26 15:09:46 -0700 |
| commit | 200f6100a9f9fc71273aeb6aceac4430f3437195 (patch) | |
| tree | 4ad58de4794de64311c27fdaf348036ddca4bc69 /src/backend/executor/nodeSubplan.c | |
| parent | 56788d2156fc32bd5737e7ac716d70e6a269b7bc (diff) | |
| download | postgresql-200f6100a9f9fc71273aeb6aceac4430f3437195.tar.gz postgresql-200f6100a9f9fc71273aeb6aceac4430f3437195.zip | |
Fix LookupTupleHashEntryHash() pipeline-stall issue.
Refactor hash lookups in nodeAgg.c to improve performance.
Author: Andres Freund and Jeff Davis
Discussion: https://postgr.es/m/20200612213715.op4ye4q7gktqvpuo%40alap3.anarazel.de
Backpatch-through: 13
Diffstat (limited to 'src/backend/executor/nodeSubplan.c')
| -rw-r--r-- | src/backend/executor/nodeSubplan.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/backend/executor/nodeSubplan.c b/src/backend/executor/nodeSubplan.c index 298b7757f57..38c2fc0b50b 100644 --- a/src/backend/executor/nodeSubplan.c +++ b/src/backend/executor/nodeSubplan.c @@ -595,12 +595,12 @@ buildSubPlanHash(SubPlanState *node, ExprContext *econtext) */ if (slotNoNulls(slot)) { - (void) LookupTupleHashEntry(node->hashtable, slot, &isnew); + (void) LookupTupleHashEntry(node->hashtable, slot, &isnew, NULL); node->havehashrows = true; } else if (node->hashnulls) { - (void) LookupTupleHashEntry(node->hashnulls, slot, &isnew); + (void) LookupTupleHashEntry(node->hashnulls, slot, &isnew, NULL); node->havenullrows = true; } |