forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit7b544c4
committed
Doc: note that statement-level view triggers require an INSTEAD OF trigger.
If a view lacks an INSTEAD OF trigger, DML on it can only work by rewritingthe command into a command on the underlying base table(s). Then we willfire triggers attached to those table(s), not those for the view. Thisseems appropriate from a consistency standpoint, but nowhere was thebehavior explicitly documented, so let's do that.There was some discussion of throwing an error or warning if a statementtrigger is created on a view without creating a row INSTEAD OF trigger.But a simple implementation of that would result in dump/restore orderinghazards. Given that it's been like this all along, and we hadn't hearda complaint till now, a documentation improvement seems sufficient.Per bug #15106 from Pu Qun. Back-patch to all supported branches.Discussion:https://postgr.es/m/152083391168.1215.16892140713507052796@wrigleys.postgresql.org1 parent59743de commit7b544c4
2 files changed
+22
-3
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
409 | 409 | | |
410 | 410 | | |
411 | 411 | | |
| 412 | + | |
| 413 | + | |
| 414 | + | |
| 415 | + | |
| 416 | + | |
| 417 | + | |
| 418 | + | |
| 419 | + | |
| 420 | + | |
| 421 | + | |
| 422 | + | |
| 423 | + | |
| 424 | + | |
412 | 425 | | |
413 | 426 | | |
414 | 427 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
56 | 56 | | |
57 | 57 | | |
58 | 58 | | |
59 | | - | |
| 59 | + | |
| 60 | + | |
60 | 61 | | |
61 | 62 | | |
62 | | - | |
63 | | - | |
| 63 | + | |
| 64 | + | |
64 | 65 | | |
65 | 66 | | |
66 | 67 | | |
67 | 68 | | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
68 | 74 | | |
69 | 75 | | |
70 | 76 | | |
| |||
0 commit comments
Comments
(0)