forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit840ff5f
committed
Get rid of recursion-marker values in enum AlterTableType
During ALTER TABLE execution, when prep-time handling of subcommands ofcertain types determine that execution-time handling requires recursion,they signal this by changing the subcommand type to a special value.This can be done in a simpler way by using a separate flag introduced bycommitec0925c, so do that.Catversion bumped. It's not clear to me that ALTER TABLE subcommandsare stored anywhere in catalogs (CREATE FUNCTION rejects it in BEGINATOMIC function bodies), but we do have both write and read support forthem, so be safe.Discussion:https://postgr.es/m/20220929090033.zxuaezcdwh2fgfjb@alvherre.pgsql1 parent9d0cf57 commit840ff5f
File tree
5 files changed
+18
-78
lines changed- src
- backend
- commands
- parser
- include
- catalog
- nodes
- test/modules/test_ddl_deparse
5 files changed
+18
-78
lines changedLines changed: 13 additions & 54 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
4070 | 4070 |
| |
4071 | 4071 |
| |
4072 | 4072 |
| |
4073 |
| - | |
4074 |
| - | |
| 4073 | + | |
4075 | 4074 |
| |
4076 | 4075 |
| |
4077 | 4076 |
| |
| |||
4276 | 4275 |
| |
4277 | 4276 |
| |
4278 | 4277 |
| |
4279 |
| - | |
4280 | 4278 |
| |
4281 | 4279 |
| |
4282 | 4280 |
| |
| |||
4628 | 4626 |
| |
4629 | 4627 |
| |
4630 | 4628 |
| |
4631 |
| - | |
| 4629 | + | |
4632 | 4630 |
| |
4633 | 4631 |
| |
4634 | 4632 |
| |
| |||
4643 | 4641 |
| |
4644 | 4642 |
| |
4645 | 4643 |
| |
4646 |
| - | |
| 4644 | + | |
4647 | 4645 |
| |
4648 | 4646 |
| |
4649 | 4647 |
| |
| |||
4765 | 4763 |
| |
4766 | 4764 |
| |
4767 | 4765 |
| |
4768 |
| - | |
| 4766 | + | |
4769 | 4767 |
| |
4770 | 4768 |
| |
4771 | 4769 |
| |
| |||
4930 | 4928 |
| |
4931 | 4929 |
| |
4932 | 4930 |
| |
4933 |
| - | |
4934 |
| - | |
4935 |
| - | |
4936 |
| - | |
4937 |
| - | |
4938 |
| - | |
| 4931 | + | |
4939 | 4932 |
| |
4940 | 4933 |
| |
4941 | 4934 |
| |
| |||
4989 | 4982 |
| |
4990 | 4983 |
| |
4991 | 4984 |
| |
4992 |
| - | |
4993 |
| - | |
4994 |
| - | |
4995 |
| - | |
4996 |
| - | |
4997 |
| - | |
4998 |
| - | |
| 4985 | + | |
4999 | 4986 |
| |
5000 | 4987 |
| |
5001 | 4988 |
| |
| |||
5015 | 5002 |
| |
5016 | 5003 |
| |
5017 | 5004 |
| |
5018 |
| - | |
| 5005 | + | |
5019 | 5006 |
| |
5020 | 5007 |
| |
5021 | 5008 |
| |
5022 | 5009 |
| |
5023 | 5010 |
| |
5024 | 5011 |
| |
5025 |
| - | |
5026 |
| - | |
5027 |
| - | |
5028 |
| - | |
5029 |
| - | |
5030 |
| - | |
5031 |
| - | |
5032 |
| - | |
5033 |
| - | |
5034 |
| - | |
5035 |
| - | |
5036 |
| - | |
5037 |
| - | |
5038 |
| - | |
| 5012 | + | |
5039 | 5013 |
| |
5040 | 5014 |
| |
5041 | 5015 |
| |
| |||
5060 | 5034 |
| |
5061 | 5035 |
| |
5062 | 5036 |
| |
5063 |
| - | |
5064 |
| - | |
5065 |
| - | |
5066 |
| - | |
5067 |
| - | |
5068 |
| - | |
| 5037 | + | |
5069 | 5038 |
| |
5070 | 5039 |
| |
5071 | 5040 |
| |
5072 | 5041 |
| |
5073 |
| - | |
5074 |
| - | |
5075 |
| - | |
5076 |
| - | |
5077 |
| - | |
5078 |
| - | |
| 5042 | + | |
5079 | 5043 |
| |
5080 | 5044 |
| |
5081 | 5045 |
| |
| |||
5351 | 5315 |
| |
5352 | 5316 |
| |
5353 | 5317 |
| |
5354 |
| - | |
| 5318 | + | |
5355 | 5319 |
| |
5356 | 5320 |
| |
5357 | 5321 |
| |
| |||
6110 | 6074 |
| |
6111 | 6075 |
| |
6112 | 6076 |
| |
6113 |
| - | |
6114 | 6077 |
| |
6115 | 6078 |
| |
6116 | 6079 |
| |
| |||
6135 | 6098 |
| |
6136 | 6099 |
| |
6137 | 6100 |
| |
6138 |
| - | |
6139 | 6101 |
| |
6140 | 6102 |
| |
6141 | 6103 |
| |
6142 | 6104 |
| |
6143 | 6105 |
| |
6144 |
| - | |
6145 | 6106 |
| |
6146 | 6107 |
| |
6147 | 6108 |
| |
6148 | 6109 |
| |
6149 | 6110 |
| |
6150 | 6111 |
| |
6151 | 6112 |
| |
6152 |
| - | |
6153 | 6113 |
| |
6154 | 6114 |
| |
6155 |
| - | |
6156 | 6115 |
| |
6157 | 6116 |
| |
6158 | 6117 |
| |
| |||
6671 | 6630 |
| |
6672 | 6631 |
| |
6673 | 6632 |
| |
6674 |
| - | |
| 6633 | + | |
6675 | 6634 |
| |
6676 | 6635 |
| |
6677 | 6636 |
| |
| |||
8376 | 8335 |
| |
8377 | 8336 |
| |
8378 | 8337 |
| |
8379 |
| - | |
| 8338 | + | |
8380 | 8339 |
| |
8381 | 8340 |
| |
8382 | 8341 |
| |
|
Lines changed: 0 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3362 | 3362 |
| |
3363 | 3363 |
| |
3364 | 3364 |
| |
3365 |
| - | |
3366 | 3365 |
| |
3367 | 3366 |
| |
3368 | 3367 |
| |
| |||
3386 | 3385 |
| |
3387 | 3386 |
| |
3388 | 3387 |
| |
3389 |
| - | |
3390 | 3388 |
| |
3391 | 3389 |
| |
3392 | 3390 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
57 | 57 |
| |
58 | 58 |
| |
59 | 59 |
| |
60 |
| - | |
| 60 | + | |
61 | 61 |
| |
62 | 62 |
|
Lines changed: 0 additions & 5 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1979 | 1979 |
| |
1980 | 1980 |
| |
1981 | 1981 |
| |
1982 |
| - | |
1983 | 1982 |
| |
1984 | 1983 |
| |
1985 | 1984 |
| |
| |||
1993 | 1992 |
| |
1994 | 1993 |
| |
1995 | 1994 |
| |
1996 |
| - | |
1997 | 1995 |
| |
1998 | 1996 |
| |
1999 | 1997 |
| |
2000 |
| - | |
2001 | 1998 |
| |
2002 | 1999 |
| |
2003 | 2000 |
| |
2004 | 2001 |
| |
2005 |
| - | |
2006 | 2002 |
| |
2007 | 2003 |
| |
2008 |
| - | |
2009 | 2004 |
| |
2010 | 2005 |
| |
2011 | 2006 |
| |
|
Lines changed: 4 additions & 16 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
114 | 114 |
| |
115 | 115 |
| |
116 | 116 |
| |
117 |
| - | |
118 |
| - | |
119 |
| - | |
120 | 117 |
| |
121 | 118 |
| |
122 | 119 |
| |
| |||
156 | 153 |
| |
157 | 154 |
| |
158 | 155 |
| |
159 |
| - | |
160 |
| - | |
161 |
| - | |
162 | 156 |
| |
163 | 157 |
| |
164 | 158 |
| |
| |||
168 | 162 |
| |
169 | 163 |
| |
170 | 164 |
| |
171 |
| - | |
172 |
| - | |
173 |
| - | |
174 | 165 |
| |
175 | 166 |
| |
176 | 167 |
| |
| |||
183 | 174 |
| |
184 | 175 |
| |
185 | 176 |
| |
186 |
| - | |
187 |
| - | |
188 |
| - | |
189 | 177 |
| |
190 | 178 |
| |
191 | 179 |
| |
192 | 180 |
| |
193 | 181 |
| |
194 | 182 |
| |
195 |
| - | |
196 |
| - | |
197 |
| - | |
198 | 183 |
| |
199 | 184 |
| |
200 | 185 |
| |
| |||
326 | 311 |
| |
327 | 312 |
| |
328 | 313 |
| |
329 |
| - | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
330 | 318 |
| |
331 | 319 |
| |
332 | 320 |
| |
|
0 commit comments
Comments
(0)