- Notifications
You must be signed in to change notification settings - Fork5
Commit22d9ddb
committed
Fix planner to make a reasonable assumption about the amount of memory space
used by array_agg(), string_agg(), and similar aggregate functions that use"internal" as their transition datatype. The previous coding thought thistook *no* extra space, since "internal" is pass-by-value; but actually theseaggregates typically consume a great deal of space. Per bug #5608 fromItagaki Takahiro, and fix suggestion from Hitoshi Harada.Back-patch to 8.4, where array_agg was introduced.1 parent41b810f commit22d9ddb
1 file changed
+13
-1
lines changedLines changed: 13 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
8 | 8 |
| |
9 | 9 |
| |
10 | 10 |
| |
11 |
| - | |
| 11 | + | |
12 | 12 |
| |
13 | 13 |
| |
14 | 14 |
| |
| |||
549 | 549 |
| |
550 | 550 |
| |
551 | 551 |
| |
| 552 | + | |
| 553 | + | |
| 554 | + | |
| 555 | + | |
| 556 | + | |
| 557 | + | |
| 558 | + | |
| 559 | + | |
| 560 | + | |
| 561 | + | |
| 562 | + | |
| 563 | + | |
552 | 564 |
| |
553 | 565 |
| |
554 | 566 |
| |
|
0 commit comments
Comments
(0)