forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitec38694
committed
Move PartitioPruneInfo out of plan nodes into PlannedStmt
The planner will now add a given PartitioPruneInfo toPlannedStmt.partPruneInfos instead of directly to theAppend/MergeAppend plan node. What gets set instead in thelatter is an index field which points to the list elementof PlannedStmt.partPruneInfos containing the PartitioPruneInfobelonging to the plan node.A later commit will make AcquireExecutorLocks() do the initialpartition pruning to determine a minimal set of partitions to belocked when validating a plan tree and it will need to consult thePartitioPruneInfos referenced therein to do so. It would be betterfor the PartitioPruneInfos to be accessible directly than requiringa walk of the plan tree to find them, which is easier when it can bedone by simply iterating over PlannedStmt.partPruneInfos.Author: Amit Langote <amitlangote09@gmail.com>Discussion:https://postgr.es/m/CA+HiwqFGkMSge6TgC9KQzde0ohpAycLQuV7ooitEEpbKB0O_mg@mail.gmail.com1 parentde867c9 commitec38694
File tree
16 files changed
+114
-63
lines changed- src
- backend
- executor
- optimizer/plan
- partitioning
- include
- catalog
- executor
- nodes
- partitioning
16 files changed
+114
-63
lines changedLines changed: 1 addition & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
825 | 825 |
| |
826 | 826 |
| |
827 | 827 |
| |
| 828 | + | |
828 | 829 |
| |
829 | 830 |
| |
830 | 831 |
| |
|
Lines changed: 1 addition & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
183 | 183 |
| |
184 | 184 |
| |
185 | 185 |
| |
| 186 | + | |
186 | 187 |
| |
187 | 188 |
| |
188 | 189 |
| |
|
Lines changed: 17 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1791 | 1791 |
| |
1792 | 1792 |
| |
1793 | 1793 |
| |
| 1794 | + | |
| 1795 | + | |
| 1796 | + | |
1794 | 1797 |
| |
1795 | 1798 |
| |
1796 | 1799 |
| |
| |||
1803 | 1806 |
| |
1804 | 1807 |
| |
1805 | 1808 |
| |
1806 |
| - | |
| 1809 | + | |
| 1810 | + | |
1807 | 1811 |
| |
1808 | 1812 |
| |
1809 | 1813 |
| |
1810 | 1814 |
| |
| 1815 | + | |
| 1816 | + | |
| 1817 | + | |
| 1818 | + | |
| 1819 | + | |
| 1820 | + | |
| 1821 | + | |
| 1822 | + | |
| 1823 | + | |
| 1824 | + | |
| 1825 | + | |
| 1826 | + | |
1811 | 1827 |
| |
1812 | 1828 |
| |
1813 | 1829 |
| |
|
Lines changed: 1 addition & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
119 | 119 |
| |
120 | 120 |
| |
121 | 121 |
| |
| 122 | + | |
122 | 123 |
| |
123 | 124 |
| |
124 | 125 |
| |
|
Lines changed: 3 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
134 | 134 |
| |
135 | 135 |
| |
136 | 136 |
| |
137 |
| - | |
| 137 | + | |
138 | 138 |
| |
139 | 139 |
| |
140 | 140 |
| |
| |||
145 | 145 |
| |
146 | 146 |
| |
147 | 147 |
| |
148 |
| - | |
| 148 | + | |
| 149 | + | |
149 | 150 |
| |
150 | 151 |
| |
151 | 152 |
| |
|
Lines changed: 3 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
82 | 82 |
| |
83 | 83 |
| |
84 | 84 |
| |
85 |
| - | |
| 85 | + | |
86 | 86 |
| |
87 | 87 |
| |
88 | 88 |
| |
| |||
93 | 93 |
| |
94 | 94 |
| |
95 | 95 |
| |
96 |
| - | |
| 96 | + | |
| 97 | + | |
97 | 98 |
| |
98 | 99 |
| |
99 | 100 |
| |
|
Lines changed: 13 additions & 11 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1203 | 1203 |
| |
1204 | 1204 |
| |
1205 | 1205 |
| |
1206 |
| - | |
1207 | 1206 |
| |
1208 | 1207 |
| |
1209 | 1208 |
| |
| |||
1354 | 1353 |
| |
1355 | 1354 |
| |
1356 | 1355 |
| |
| 1356 | + | |
| 1357 | + | |
| 1358 | + | |
1357 | 1359 |
| |
1358 | 1360 |
| |
1359 | 1361 |
| |
| |||
1377 | 1379 |
| |
1378 | 1380 |
| |
1379 | 1381 |
| |
1380 |
| - | |
1381 |
| - | |
1382 |
| - | |
1383 |
| - | |
| 1382 | + | |
| 1383 | + | |
| 1384 | + | |
1384 | 1385 |
| |
1385 | 1386 |
| |
1386 | 1387 |
| |
1387 | 1388 |
| |
1388 | 1389 |
| |
1389 |
| - | |
1390 | 1390 |
| |
1391 | 1391 |
| |
1392 | 1392 |
| |
| |||
1425 | 1425 |
| |
1426 | 1426 |
| |
1427 | 1427 |
| |
1428 |
| - | |
1429 | 1428 |
| |
1430 | 1429 |
| |
1431 | 1430 |
| |
| |||
1518 | 1517 |
| |
1519 | 1518 |
| |
1520 | 1519 |
| |
| 1520 | + | |
| 1521 | + | |
| 1522 | + | |
1521 | 1523 |
| |
1522 | 1524 |
| |
1523 | 1525 |
| |
| |||
1541 | 1543 |
| |
1542 | 1544 |
| |
1543 | 1545 |
| |
1544 |
| - | |
1545 |
| - | |
1546 |
| - | |
| 1546 | + | |
| 1547 | + | |
| 1548 | + | |
1547 | 1549 |
| |
1548 | 1550 |
| |
1549 | 1551 |
| |
1550 |
| - | |
| 1552 | + | |
1551 | 1553 |
| |
1552 | 1554 |
| |
1553 | 1555 |
| |
|
Lines changed: 1 addition & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
519 | 519 |
| |
520 | 520 |
| |
521 | 521 |
| |
| 522 | + | |
522 | 523 |
| |
523 | 524 |
| |
524 | 525 |
| |
|
Lines changed: 37 additions & 30 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
348 | 348 |
| |
349 | 349 |
| |
350 | 350 |
| |
| 351 | + | |
| 352 | + | |
| 353 | + | |
| 354 | + | |
| 355 | + | |
| 356 | + | |
| 357 | + | |
| 358 | + | |
| 359 | + | |
| 360 | + | |
| 361 | + | |
| 362 | + | |
| 363 | + | |
| 364 | + | |
| 365 | + | |
| 366 | + | |
| 367 | + | |
| 368 | + | |
| 369 | + | |
| 370 | + | |
| 371 | + | |
| 372 | + | |
| 373 | + | |
| 374 | + | |
| 375 | + | |
351 | 376 |
| |
352 | 377 |
| |
353 | 378 |
| |
| |||
1658 | 1683 |
| |
1659 | 1684 |
| |
1660 | 1685 |
| |
1661 |
| - | |
1662 |
| - | |
1663 |
| - | |
1664 |
| - | |
1665 |
| - | |
1666 |
| - | |
1667 |
| - | |
1668 |
| - | |
1669 |
| - | |
1670 |
| - | |
1671 |
| - | |
1672 |
| - | |
1673 |
| - | |
1674 |
| - | |
1675 |
| - | |
| 1686 | + | |
| 1687 | + | |
| 1688 | + | |
| 1689 | + | |
| 1690 | + | |
| 1691 | + | |
1676 | 1692 |
| |
1677 | 1693 |
| |
1678 | 1694 |
| |
| |||
1734 | 1750 |
| |
1735 | 1751 |
| |
1736 | 1752 |
| |
1737 |
| - | |
1738 |
| - | |
1739 |
| - | |
1740 |
| - | |
1741 |
| - | |
1742 |
| - | |
1743 |
| - | |
1744 |
| - | |
1745 |
| - | |
1746 |
| - | |
1747 |
| - | |
1748 |
| - | |
1749 |
| - | |
1750 |
| - | |
1751 |
| - | |
| 1753 | + | |
| 1754 | + | |
| 1755 | + | |
| 1756 | + | |
| 1757 | + | |
| 1758 | + | |
1752 | 1759 |
| |
1753 | 1760 |
| |
1754 | 1761 |
| |
|
Lines changed: 13 additions & 6 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
209 | 209 |
| |
210 | 210 |
| |
211 | 211 |
| |
212 |
| - | |
213 |
| - | |
214 |
| - | |
| 212 | + | |
| 213 | + | |
| 214 | + | |
| 215 | + | |
| 216 | + | |
| 217 | + | |
| 218 | + | |
215 | 219 |
| |
216 | 220 |
| |
217 | 221 |
| |
218 | 222 |
| |
219 | 223 |
| |
220 | 224 |
| |
221 |
| - | |
| 225 | + | |
222 | 226 |
| |
223 | 227 |
| |
224 | 228 |
| |
| |||
332 | 336 |
| |
333 | 337 |
| |
334 | 338 |
| |
335 |
| - | |
| 339 | + | |
336 | 340 |
| |
337 | 341 |
| |
338 | 342 |
| |
| 343 | + | |
339 | 344 |
| |
340 | 345 |
| |
341 | 346 |
| |
| |||
358 | 363 |
| |
359 | 364 |
| |
360 | 365 |
| |
361 |
| - | |
| 366 | + | |
| 367 | + | |
| 368 | + | |
362 | 369 |
| |
363 | 370 |
| |
364 | 371 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
57 | 57 |
| |
58 | 58 |
| |
59 | 59 |
| |
60 |
| - | |
| 60 | + | |
61 | 61 |
| |
62 | 62 |
|
Lines changed: 2 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
123 | 123 |
| |
124 | 124 |
| |
125 | 125 |
| |
126 |
| - | |
| 126 | + | |
| 127 | + | |
127 | 128 |
| |
128 | 129 |
| |
129 | 130 |
| |
130 |
| - | |
131 | 131 |
|
Lines changed: 1 addition & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
614 | 614 |
| |
615 | 615 |
| |
616 | 616 |
| |
| 617 | + | |
617 | 618 |
| |
618 | 619 |
| |
619 | 620 |
| |
|
Lines changed: 6 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
122 | 122 |
| |
123 | 123 |
| |
124 | 124 |
| |
| 125 | + | |
| 126 | + | |
| 127 | + | |
125 | 128 |
| |
126 | 129 |
| |
127 | 130 |
| |
| |||
503 | 506 |
| |
504 | 507 |
| |
505 | 508 |
| |
| 509 | + | |
| 510 | + | |
| 511 | + | |
506 | 512 |
| |
507 | 513 |
| |
508 | 514 |
| |
|
0 commit comments
Comments
(0)