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

Commit0d30e48

Browse files
committed
Fix unnecessary use of moving-aggregate mode with non-moving frame.
When a plain aggregate is used as a window function, and the windowframe start is specified as UNBOUNDED PRECEDING, the frame's headcannot move so we do not need to use moving-aggregate mode. The checkfor that was put into initialize_peragg(), failing to notice thatExecInitWindowAgg() calls that function before it's filled inwinstate->frameOptions. Since makeNode() would have zeroed the field,this didn't provoke uninitialized-value complaints, nor would theerroneous decision have resulted in more than a little inefficiency.Still, it's wrong, so move the initialization ofwinstate->frameOptions earlier to make it work properly.While here, also fix a thinko in a comment. Both errors crept in incommita9d9acb which introduced the moving-aggregate mode.Spotted by Vallimaharajan G. Back-patch to all supported branches.Discussion:https://postgr.es/m/18e7f2a5167.fe36253866818.977923893562469143@zohocorp.com
1 parent66bbad5 commit0d30e48

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

‎src/backend/executor/nodeWindowAgg.c

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -2274,6 +2274,9 @@ ExecInitWindowAgg(WindowAgg *node, EState *estate, int eflags)
22742274
winstate->ss.ps.state=estate;
22752275
winstate->ss.ps.ExecProcNode=ExecWindowAgg;
22762276

2277+
/* copy frame options to state node for easy access */
2278+
winstate->frameOptions=frameOptions;
2279+
22772280
/*
22782281
* Create expression contexts. We need two, one for per-input-tuple
22792282
* processing and one for per-output-tuple processing. We cheat a little
@@ -2501,9 +2504,6 @@ ExecInitWindowAgg(WindowAgg *node, EState *estate, int eflags)
25012504
winstate->agg_winobj=agg_winobj;
25022505
}
25032506

2504-
/* copy frame options to state node for easy access */
2505-
winstate->frameOptions=frameOptions;
2506-
25072507
/* initialize frame bound offset expressions */
25082508
winstate->startOffset=ExecInitExpr((Expr*)node->startOffset,
25092509
(PlanState*)winstate);
@@ -2654,7 +2654,7 @@ initialize_peragg(WindowAggState *winstate, WindowFunc *wfunc,
26542654

26552655
/*
26562656
* Figure out whether we want to use the moving-aggregate implementation,
2657-
* and collect the right set of fields from thepg_attribute entry.
2657+
* and collect the right set of fields from thepg_aggregate entry.
26582658
*
26592659
* It's possible that an aggregate would supply a safe moving-aggregate
26602660
* implementation and an unsafe normal one, in which case our hand is

0 commit comments

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