Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commitbdaa84e

Browse files
author
Amit Kapila
committed
Fix buffer usage stats for nodes above Gather Merge.
Commit85c9d34 addressed a similar problem for Gather and GatherMerge nodes but forgot to account for nodes above parallel nodes. Thisstill works for nodes above Gather node because we shut down the workersfor Gather node as soon as there are no more tuples. We can do a similarthing for Gather Merge as well but it seems better to account for statsduring nodes shutdown after completing the execution.Reported-by: Stéphane Lorek, Jehan-Guillaume de RorthaisAuthor: Jehan-Guillaume de Rorthais <jgdr@dalibo.com>Reviewed-by: Amit KapilaBackpatch-through: 10, where it was introducedDiscussion:https://postgr.es/m/20200718160206.584532a2@firost
1 parent3d4a778 commitbdaa84e

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

‎src/backend/executor/execProcnode.c‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -746,8 +746,6 @@ ExecShutdownNode(PlanState *node)
746746

747747
check_stack_depth();
748748

749-
planstate_tree_walker(node,ExecShutdownNode,NULL);
750-
751749
/*
752750
* Treat the node as running while we shut it down, but only if it's run
753751
* at least once already. We don't expect much CPU consumption during
@@ -761,6 +759,8 @@ ExecShutdownNode(PlanState *node)
761759
if (node->instrument&&node->instrument->running)
762760
InstrStartNode(node->instrument);
763761

762+
planstate_tree_walker(node,ExecShutdownNode,NULL);
763+
764764
switch (nodeTag(node))
765765
{
766766
caseT_GatherState:

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp