forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit3c49d46
committed
Disallow NAMEDTUPLESTORE RTEs in stored views, rules, etc.
A named tuplestore is necessarily a transient object, so it makesno sense to reference one in a persistent object such as a view.We didn't previously prevent that, with the result that if youtried you would get some weird failure about how the executorcouldn't find the tuplestore.We can mechanize a check for this case cheaply by making dependencyextraction complain if it comes across such an RTE. This is aplausible way of dealing with it since part of the problem is that wehave no way to make a pg_depend representation of a named tuplestore.Report and fix by Yugo Nagata. Although this is an old problem,it's a very weird corner case and there have been no reports fromend users. So it seems sufficient to fix it in master.Discussion:https://postgr.es/m/20240726160714.e74d0db579f2c017e1ca0b7e@sraoss.co.jp1 parentb20fe54 commit3c49d46
File tree
3 files changed
+54
-0
lines changed- src
- backend/catalog
- test/regress
- expected
- sql
3 files changed
+54
-0
lines changedLines changed: 15 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2191 | 2191 |
| |
2192 | 2192 |
| |
2193 | 2193 |
| |
| 2194 | + | |
| 2195 | + | |
| 2196 | + | |
| 2197 | + | |
| 2198 | + | |
| 2199 | + | |
| 2200 | + | |
| 2201 | + | |
| 2202 | + | |
| 2203 | + | |
| 2204 | + | |
| 2205 | + | |
| 2206 | + | |
| 2207 | + | |
2194 | 2208 |
| |
| 2209 | + | |
2195 | 2210 |
| |
2196 | 2211 |
| |
2197 | 2212 |
| |
|
Lines changed: 20 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3315 | 3315 |
| |
3316 | 3316 |
| |
3317 | 3317 |
| |
| 3318 | + | |
| 3319 | + | |
| 3320 | + | |
| 3321 | + | |
| 3322 | + | |
| 3323 | + | |
| 3324 | + | |
| 3325 | + | |
| 3326 | + | |
| 3327 | + | |
| 3328 | + | |
| 3329 | + | |
| 3330 | + | |
| 3331 | + | |
| 3332 | + | |
| 3333 | + | |
| 3334 | + | |
| 3335 | + | |
| 3336 | + | |
| 3337 | + | |
3318 | 3338 |
| |
3319 | 3339 |
| |
3320 | 3340 |
| |
|
Lines changed: 19 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2434 | 2434 |
| |
2435 | 2435 |
| |
2436 | 2436 |
| |
| 2437 | + | |
| 2438 | + | |
| 2439 | + | |
| 2440 | + | |
| 2441 | + | |
| 2442 | + | |
| 2443 | + | |
| 2444 | + | |
| 2445 | + | |
| 2446 | + | |
| 2447 | + | |
| 2448 | + | |
| 2449 | + | |
| 2450 | + | |
| 2451 | + | |
| 2452 | + | |
| 2453 | + | |
| 2454 | + | |
| 2455 | + | |
2437 | 2456 |
| |
2438 | 2457 |
| |
2439 | 2458 |
| |
|
0 commit comments
Comments
(0)