forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitfed10d4
committed
Properly mark pg_stat_get_subscription() as returning a set.
The initial catalog data for this function failed to set proretsetor provide a prorows estimate. It accidentally worked anyway wheninvoked in the FROM clause, because the executor isn't too pickyabout this; but the planner didn't expect the function to returnmultiple rows, which could lead to bad plans. Also the functionwould fail if invoked in the SELECT list.We can't easily back-patch this fix, but fortunately the bug'sconsequences aren't awful in most cases. Getting this right ismainly an exercise in future-proofing.Discussion:https://postgr.es/m/1636062.1615141782@sss.pgh.pa.us1 parent5c06abb commitfed10d4
2 files changed
+4
-3
lines changedOriginal file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
53 | 53 |
| |
54 | 54 |
| |
55 | 55 |
| |
56 |
| - | |
| 56 | + | |
57 | 57 |
| |
58 | 58 |
|
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
5287 | 5287 |
| |
5288 | 5288 |
| |
5289 | 5289 |
| |
5290 |
| - | |
5291 |
| - | |
| 5290 | + | |
| 5291 | + | |
| 5292 | + | |
5292 | 5293 |
| |
5293 | 5294 |
| |
5294 | 5295 |
| |
|
0 commit comments
Comments
(0)