forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitb7d35d3
committed
Remove some unnecessary fields from executor nodes.
JsonExprState.input_finfo is only assigned to, never read, andit's really fairly useless since the value can be gotten out ofthe adjacent input_fcinfo field. Let's remove it before someonestarts to depend on it.While here, also remove TidScanState.tss_htup and AggState.combinedproj,which are referenced nowhere. Those should have been removed by thecommits that caused them to become disused, but were not.I don't think a catversion bump is necessary here, since plan treesare never stored on disk.Matthias van de MeentDiscussion:https://postgr.es/m/CAEze2WjsY4d0TBymLNGK4zpttUcg_YZaTjyWz2VfDUV6YH8wXQ@mail.gmail.com1 parent598e011 commitb7d35d3
2 files changed
+0
-5
lines changedLines changed: 0 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
4393 | 4393 |
| |
4394 | 4394 |
| |
4395 | 4395 |
| |
4396 |
| - | |
4397 | 4396 |
| |
4398 | 4397 |
| |
4399 | 4398 |
| |
|
Lines changed: 0 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1070 | 1070 |
| |
1071 | 1071 |
| |
1072 | 1072 |
| |
1073 |
| - | |
1074 | 1073 |
| |
1075 | 1074 |
| |
1076 | 1075 |
| |
| |||
1835 | 1834 |
| |
1836 | 1835 |
| |
1837 | 1836 |
| |
1838 |
| - | |
1839 | 1837 |
| |
1840 | 1838 |
| |
1841 | 1839 |
| |
| |||
1846 | 1844 |
| |
1847 | 1845 |
| |
1848 | 1846 |
| |
1849 |
| - | |
1850 | 1847 |
| |
1851 | 1848 |
| |
1852 | 1849 |
| |
| |||
2530 | 2527 |
| |
2531 | 2528 |
| |
2532 | 2529 |
| |
2533 |
| - | |
2534 | 2530 |
| |
2535 | 2531 |
| |
2536 | 2532 |
| |
|
0 commit comments
Comments
(0)