forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitf6dba10
committed
First phase of implementing hash-based grouping/aggregation. An AGG plan
node now does its own grouping of the input rows, and has no need for apreceding GROUP node in the plan pipeline. This allows elimination ofthe misnamed tuplePerGroup option for GROUP, and actually saves more codein nodeGroup.c than it costs in nodeAgg.c, as well as being presumablyfaster. Restructure the API of query_planner so that we do not commit tousing a sorted or unsorted plan in query_planner; instead grouping_plannermakes the decision. (Right now it isn't any smarter than query_plannerwas, but that will change as soon as it has the option to select a hash-based aggregation step.) Despite all the hackery, no initdb needed sinceonly in-memory node types changed.1 parenta8c18b9 commitf6dba10
File tree
22 files changed
+797
-908
lines changed- src
- backend
- commands
- executor
- nodes
- optimizer
- geqo
- path
- plan
- util
- include
- nodes
- optimizer
22 files changed
+797
-908
lines changedLines changed: 16 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
5 | 5 |
| |
6 | 6 |
| |
7 | 7 |
| |
8 |
| - | |
| 8 | + | |
9 | 9 |
| |
10 | 10 |
| |
11 | 11 |
| |
| |||
275 | 275 |
| |
276 | 276 |
| |
277 | 277 |
| |
278 |
| - | |
| 278 | + | |
| 279 | + | |
| 280 | + | |
| 281 | + | |
| 282 | + | |
| 283 | + | |
| 284 | + | |
| 285 | + | |
| 286 | + | |
| 287 | + | |
| 288 | + | |
| 289 | + | |
| 290 | + | |
| 291 | + | |
| 292 | + | |
279 | 293 |
| |
280 | 294 |
| |
281 | 295 |
| |
|
0 commit comments
Comments
(0)