- Notifications
You must be signed in to change notification settings - Fork28
Commitc759395
committed
Code review for Parallel Append.
- Remove unnecessary #include mistakenly added in execnodes.h.- Fix mistake in comment in choose_next_subplan_for_leader.- Adjust row estimates in cost_append for a possibly-different parallel divisor.- Clamp row estimates in cost_append after operations that may not produce integers.Amit Kapila, with cosmetic adjustments by me.Discussion:http://postgr.es/m/CAA4eK1+qcbeai3coPpRW=GFCzFeLUsuY4T-AKHqMjxpEGZBPQg@mail.gmail.com1 parent934c798 commitc759395
File tree
3 files changed
+15
-9
lines changed- src
- backend
- executor
- optimizer/path
- include/nodes
3 files changed
+15
-9
lines changedLines changed: 3 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
446 | 446 |
| |
447 | 447 |
| |
448 | 448 |
| |
449 |
| - | |
450 |
| - | |
451 |
| - | |
452 |
| - | |
| 449 | + | |
| 450 | + | |
| 451 | + | |
453 | 452 |
| |
454 | 453 |
| |
455 | 454 |
| |
|
Lines changed: 12 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1883 | 1883 |
| |
1884 | 1884 |
| |
1885 | 1885 |
| |
1886 |
| - | |
1887 |
| - | |
1888 |
| - | |
| 1886 | + | |
| 1887 | + | |
| 1888 | + | |
| 1889 | + | |
| 1890 | + | |
1889 | 1891 |
| |
1890 | 1892 |
| |
1891 | 1893 |
| |
1892 | 1894 |
| |
1893 | 1895 |
| |
1894 |
| - | |
| 1896 | + | |
| 1897 | + | |
| 1898 | + | |
| 1899 | + | |
| 1900 | + | |
1895 | 1901 |
| |
1896 | 1902 |
| |
1897 | 1903 |
| |
| 1904 | + | |
| 1905 | + | |
1898 | 1906 |
| |
1899 | 1907 |
| |
1900 | 1908 |
| |
|
Lines changed: 0 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
21 | 21 |
| |
22 | 22 |
| |
23 | 23 |
| |
24 |
| - | |
25 | 24 |
| |
26 | 25 |
| |
27 | 26 |
| |
|
0 commit comments
Comments
(0)