forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitee95532
committed
Handle WindowClause.runCondition in tree walker/mutator functions.
Commit9d9c02c, which added the notion of a "run condition" forwindow functions, neglected to teach nodeFuncs.c to process the newfield. Remarkably, that doesn't seem to have had any ill effectsbefore we invented Var.varnullingrels, but now it can cause visiblefailures in join-removal scenarios.I have no faith that there's not reachable problems in v15 too,so back-patch the code change to v15 where9d9c02c came in.The test case seems irrelevant to v15, though.Per bug #18277 from Zuming Jiang. Diagnosis and patch byRichard Guo.Discussion:https://postgr.es/m/18277-089ead83b329a2fd@postgresql.org1 parent65ee2ed commitee95532
File tree
3 files changed
+41
-0
lines changed- src
- backend/nodes
- test/regress
- expected
- sql
3 files changed
+41
-0
lines changedLines changed: 6 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2283 | 2283 |
| |
2284 | 2284 |
| |
2285 | 2285 |
| |
| 2286 | + | |
| 2287 | + | |
2286 | 2288 |
| |
2287 | 2289 |
| |
2288 | 2290 |
| |
| |||
2627 | 2629 |
| |
2628 | 2630 |
| |
2629 | 2631 |
| |
| 2632 | + | |
| 2633 | + | |
2630 | 2634 |
| |
2631 | 2635 |
| |
2632 | 2636 |
| |
| |||
3312 | 3316 |
| |
3313 | 3317 |
| |
3314 | 3318 |
| |
| 3319 | + | |
3315 | 3320 |
| |
3316 | 3321 |
| |
3317 | 3322 |
| |
| |||
3641 | 3646 |
| |
3642 | 3647 |
| |
3643 | 3648 |
| |
| 3649 | + | |
3644 | 3650 |
| |
3645 | 3651 |
| |
3646 | 3652 |
| |
|
Lines changed: 26 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3804 | 3804 |
| |
3805 | 3805 |
| |
3806 | 3806 |
| |
| 3807 | + | |
| 3808 | + | |
| 3809 | + | |
| 3810 | + | |
| 3811 | + | |
| 3812 | + | |
| 3813 | + | |
| 3814 | + | |
| 3815 | + | |
| 3816 | + | |
| 3817 | + | |
| 3818 | + | |
| 3819 | + | |
| 3820 | + | |
| 3821 | + | |
| 3822 | + | |
| 3823 | + | |
| 3824 | + | |
| 3825 | + | |
| 3826 | + | |
| 3827 | + | |
| 3828 | + | |
| 3829 | + | |
| 3830 | + | |
| 3831 | + | |
| 3832 | + | |
3807 | 3833 |
| |
3808 | 3834 |
| |
3809 | 3835 |
| |
|
Lines changed: 9 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1236 | 1236 |
| |
1237 | 1237 |
| |
1238 | 1238 |
| |
| 1239 | + | |
| 1240 | + | |
| 1241 | + | |
| 1242 | + | |
| 1243 | + | |
| 1244 | + | |
| 1245 | + | |
| 1246 | + | |
| 1247 | + | |
1239 | 1248 |
| |
1240 | 1249 |
| |
1241 | 1250 |
| |
|
0 commit comments
Comments
(0)