forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit1b7f3cc
committed
This patch implements FOR EACH STATEMENT triggers, per my email to
-hackers a couple days ago.Notes/caveats: - added regression tests for the new functionality, all regression tests pass on my machine - added pg_dump support - updated PL/PgSQL to support per-statement triggers; didn't look at the other procedural languages. - there's (even) more code duplication in trigger.c than there was previously. Any suggestions on how to refactor the ExecXXXTriggers() functions to reuse more code would be welcome -- I took a brief look at it, but couldn't see an easy way to do it (there are several subtly-different versions of the code in question) - updated the documentation. I also took the liberty of removing a big chunk of duplicated syntax documentation in the Programmer's Guide on triggers, and moving that information to the CREATE TRIGGER reference page. - I also included some spelling fixes and similar small cleanups I noticed while making the changes. If you'd like me to split those into a separate patch, let me know.Neil Conway1 parentea29b32 commit1b7f3cc
File tree
24 files changed
+702
-411
lines changed- doc/src/sgml
- ref
- src
- backend
- access/transam
- commands
- executor
- nodes
- parser
- utils/adt
- bin/pg_dump
- include
- commands
- nodes
- utils
- interfaces/python
- pl/plpgsql/src
- test/regress
- expected
- sql
24 files changed
+702
-411
lines changedLines changed: 55 additions & 49 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1 | 1 |
| |
2 |
| - | |
| 2 | + | |
3 | 3 |
| |
4 | 4 |
| |
5 | 5 |
| |
| |||
674 | 674 |
| |
675 | 675 |
| |
676 | 676 |
| |
677 |
| - | |
678 |
| - | |
679 |
| - | |
680 |
| - | |
681 |
| - | |
682 |
| - | |
683 |
| - | |
684 |
| - | |
685 |
| - | |
| 677 | + | |
| 678 | + | |
| 679 | + | |
| 680 | + | |
| 681 | + | |
| 682 | + | |
| 683 | + | |
| 684 | + | |
| 685 | + | |
686 | 686 |
| |
687 | 687 |
| |
688 | 688 |
| |
689 |
| - | |
690 |
| - | |
| 689 | + | |
| 690 | + | |
691 | 691 |
| |
692 | 692 |
| |
693 |
| - | |
694 |
| - | |
| 693 | + | |
| 694 | + | |
| 695 | + | |
695 | 696 |
| |
696 | 697 |
| |
697 | 698 |
| |
| |||
1100 | 1101 |
| |
1101 | 1102 |
| |
1102 | 1103 |
| |
1103 |
| - | |
1104 |
| - | |
| 1104 | + | |
| 1105 | + | |
| 1106 | + | |
1105 | 1107 |
| |
1106 | 1108 |
| |
1107 | 1109 |
| |
1108 | 1110 |
| |
1109 |
| - | |
| 1111 | + | |
1110 | 1112 |
| |
1111 | 1113 |
| |
1112 | 1114 |
| |
1113 | 1115 |
| |
1114 | 1116 |
| |
1115 | 1117 |
| |
1116 |
| - | |
1117 |
| - | |
1118 |
| - | |
| 1118 | + | |
| 1119 | + | |
| 1120 | + | |
1119 | 1121 |
| |
1120 | 1122 |
| |
1121 | 1123 |
| |
1122 | 1124 |
| |
1123 |
| - | |
| 1125 | + | |
1124 | 1126 |
| |
1125 | 1127 |
| |
1126 | 1128 |
| |
1127 | 1129 |
| |
1128 | 1130 |
| |
1129 |
| - | |
1130 |
| - | |
1131 |
| - | |
1132 |
| - | |
1133 |
| - | |
1134 |
| - | |
1135 |
| - | |
1136 |
| - | |
1137 |
| - | |
| 1131 | + | |
| 1132 | + | |
| 1133 | + | |
| 1134 | + | |
| 1135 | + | |
| 1136 | + | |
| 1137 | + | |
| 1138 | + | |
| 1139 | + | |
| 1140 | + | |
1138 | 1141 |
| |
1139 | 1142 |
| |
1140 | 1143 |
| |
| |||
1975 | 1978 |
| |
1976 | 1979 |
| |
1977 | 1980 |
| |
1978 |
| - | |
| 1981 | + | |
1979 | 1982 |
| |
1980 | 1983 |
| |
1981 | 1984 |
| |
| |||
1992 | 1995 |
| |
1993 | 1996 |
| |
1994 | 1997 |
| |
1995 |
| - | |
1996 |
| - | |
| 1998 | + | |
| 1999 | + | |
| 2000 | + | |
1997 | 2001 |
| |
1998 | 2002 |
| |
1999 | 2003 |
| |
| |||
2002 | 2006 |
| |
2003 | 2007 |
| |
2004 | 2008 |
| |
2005 |
| - | |
2006 |
| - | |
| 2009 | + | |
| 2010 | + | |
| 2011 | + | |
2007 | 2012 |
| |
2008 | 2013 |
| |
2009 | 2014 |
| |
| |||
2098 | 2103 |
| |
2099 | 2104 |
| |
2100 | 2105 |
| |
2101 |
| - | |
2102 |
| - | |
2103 |
| - | |
2104 |
| - | |
2105 |
| - | |
2106 |
| - | |
2107 |
| - | |
2108 |
| - | |
2109 |
| - | |
2110 |
| - | |
| 2106 | + | |
| 2107 | + | |
| 2108 | + | |
| 2109 | + | |
| 2110 | + | |
2111 | 2111 |
| |
2112 | 2112 |
| |
2113 | 2113 |
| |
2114 |
| - | |
2115 |
| - | |
2116 |
| - | |
| 2114 | + | |
| 2115 | + | |
| 2116 | + | |
| 2117 | + | |
| 2118 | + | |
| 2119 | + | |
| 2120 | + | |
| 2121 | + | |
| 2122 | + | |
2117 | 2123 |
| |
2118 | 2124 |
| |
2119 | 2125 |
| |
| |||
2143 | 2149 |
| |
2144 | 2150 |
| |
2145 | 2151 |
| |
2146 |
| - | |
| 2152 | + | |
2147 | 2153 |
| |
2148 | 2154 |
| |
2149 | 2155 |
| |
|
Lines changed: 2 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
153 | 153 |
| |
154 | 154 |
| |
155 | 155 |
| |
156 |
| - | |
157 |
| - | |
| 156 | + | |
| 157 | + | |
158 | 158 |
| |
159 | 159 |
| |
160 | 160 |
| |
|
Lines changed: 62 additions & 25 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1 | 1 |
| |
2 |
| - | |
| 2 | + | |
3 | 3 |
| |
4 | 4 |
| |
5 | 5 |
| |
| |||
21 | 21 |
| |
22 | 22 |
| |
23 | 23 |
| |
24 |
| - | |
25 |
| - | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
26 | 27 |
| |
27 | 28 |
| |
28 | 29 |
| |
| |||
45 | 46 |
| |
46 | 47 |
| |
47 | 48 |
| |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
48 | 61 |
| |
49 | 62 |
| |
50 | 63 |
| |
51 | 64 |
| |
52 |
| - | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
53 | 69 |
| |
54 | 70 |
| |
55 | 71 |
| |
56 | 72 |
| |
57 | 73 |
| |
58 | 74 |
| |
59 | 75 |
| |
60 |
| - | |
| 76 | + | |
| 77 | + | |
61 | 78 |
| |
62 | 79 |
| |
63 | 80 |
| |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
64 | 96 |
| |
65 | 97 |
| |
66 | 98 |
| |
| |||
74 | 106 |
| |
75 | 107 |
| |
76 | 108 |
| |
77 |
| - | |
78 |
| - | |
79 |
| - | |
80 |
| - | |
81 |
| - | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
| 117 | + | |
82 | 118 |
| |
83 | 119 |
| |
84 | 120 |
| |
| |||
121 | 157 |
| |
122 | 158 |
| |
123 | 159 |
| |
124 |
| - | |
| 160 | + | |
125 | 161 |
| |
126 | 162 |
| |
127 | 163 |
| |
| |||
141 | 177 |
| |
142 | 178 |
| |
143 | 179 |
| |
| 180 | + | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
| 184 | + | |
| 185 | + | |
| 186 | + | |
| 187 | + | |
| 188 | + | |
| 189 | + | |
| 190 | + | |
| 191 | + | |
144 | 192 |
| |
145 | 193 |
| |
146 | 194 |
| |
147 | 195 |
| |
148 | 196 |
| |
149 | 197 |
| |
150 |
| - | |
151 |
| - | |
152 |
| - | |
| 198 | + | |
| 199 | + | |
| 200 | + | |
153 | 201 |
| |
154 | 202 |
| |
155 | 203 |
| |
| |||
176 | 224 |
| |
177 | 225 |
| |
178 | 226 |
| |
179 |
| - | |
180 |
| - | |
181 |
| - | |
182 |
| - | |
183 | 227 |
| |
184 | 228 |
| |
185 | 229 |
| |
| |||
268 | 312 |
| |
269 | 313 |
| |
270 | 314 |
| |
271 |
| - | |
272 |
| - | |
273 |
| - | |
274 |
| - | |
275 |
| - | |
276 |
| - | |
277 |
| - | |
278 | 315 |
| |
279 | 316 |
| |
280 | 317 |
| |
|
Lines changed: 2 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1 | 1 |
| |
2 |
| - | |
| 2 | + | |
3 | 3 |
| |
4 | 4 |
| |
5 | 5 |
| |
| |||
4619 | 4619 |
| |
4620 | 4620 |
| |
4621 | 4621 |
| |
4622 |
| - | |
| 4622 | + | |
4623 | 4623 |
| |
4624 | 4624 |
| |
4625 | 4625 |
| |
|
0 commit comments
Comments
(0)