- Notifications
You must be signed in to change notification settings - Fork5
Commite1ccaff
committed
Rework parsing of ConstraintAttributeSpec to improve NOT VALID handling.
The initial commit of the ALTER TABLE ADD FOREIGN KEY NOT VALID featurefailed to support labeling such constraints as deferrable. The best fixfor this seems to be to fold NOT VALID into ConstraintAttributeSpec.That's a bit more general than the documented syntax, but it allowsbetter-targeted syntax error messages.In addition, do some mostly-but-not-entirely-cosmetic code review forthe whole NOT VALID patch.1 parente3df357 commite1ccaff
File tree
9 files changed
+175
-115
lines changed- doc/src/sgml
- ref
- src
- backend
- commands
- parser
- utils/adt
- include/nodes
- interfaces/ecpg/preproc
9 files changed
+175
-115
lines changedLines changed: 2 additions & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1898 | 1898 |
| |
1899 | 1899 |
| |
1900 | 1900 |
| |
1901 |
| - | |
| 1901 | + | |
| 1902 | + | |
1902 | 1903 |
| |
1903 | 1904 |
| |
1904 | 1905 |
| |
|
Lines changed: 28 additions & 20 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
42 | 42 |
| |
43 | 43 |
| |
44 | 44 |
| |
45 |
| - | |
46 |
| - | |
47 | 45 |
| |
| 46 | + | |
48 | 47 |
| |
49 | 48 |
| |
50 | 49 |
| |
| |||
235 | 234 |
| |
236 | 235 |
| |
237 | 236 |
| |
238 |
| - | |
239 |
| - | |
| 237 | + | |
240 | 238 |
| |
241 | 239 |
| |
242 | 240 |
| |
243 |
| - | |
244 |
| - | |
245 |
| - | |
246 |
| - | |
247 |
| - | |
248 |
| - | |
249 |
| - | |
250 |
| - | |
251 |
| - | |
252 |
| - | |
253 |
| - | |
254 |
| - | |
255 |
| - | |
256 |
| - | |
257 |
| - | |
258 |
| - | |
| 241 | + | |
| 242 | + | |
| 243 | + | |
| 244 | + | |
| 245 | + | |
| 246 | + | |
| 247 | + | |
| 248 | + | |
| 249 | + | |
| 250 | + | |
| 251 | + | |
259 | 252 |
| |
260 | 253 |
| |
261 | 254 |
| |
| |||
311 | 304 |
| |
312 | 305 |
| |
313 | 306 |
| |
| 307 | + | |
| 308 | + | |
| 309 | + | |
| 310 | + | |
| 311 | + | |
| 312 | + | |
| 313 | + | |
| 314 | + | |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
| 321 | + | |
314 | 322 |
| |
315 | 323 |
| |
316 | 324 |
| |
|
Lines changed: 7 additions & 6 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
258 | 258 |
| |
259 | 259 |
| |
260 | 260 |
| |
261 |
| - | |
| 261 | + | |
262 | 262 |
| |
263 | 263 |
| |
264 | 264 |
| |
| |||
5726 | 5726 |
| |
5727 | 5727 |
| |
5728 | 5728 |
| |
5729 |
| - | |
5730 |
| - | |
5731 |
| - | |
| 5729 | + | |
| 5730 | + | |
| 5731 | + | |
5732 | 5732 |
| |
5733 | 5733 |
| |
5734 | 5734 |
| |
| |||
5755 | 5755 |
| |
5756 | 5756 |
| |
5757 | 5757 |
| |
5758 |
| - | |
| 5758 | + | |
5759 | 5759 |
| |
5760 | 5760 |
| |
5761 | 5761 |
| |
| |||
5810 | 5810 |
| |
5811 | 5811 |
| |
5812 | 5812 |
| |
5813 |
| - | |
| 5813 | + | |
5814 | 5814 |
| |
5815 | 5815 |
| |
5816 | 5816 |
| |
| |||
5830 | 5830 |
| |
5831 | 5831 |
| |
5832 | 5832 |
| |
| 5833 | + | |
5833 | 5834 |
| |
5834 | 5835 |
| |
5835 | 5836 |
| |
|
Lines changed: 2 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1006 | 1006 |
| |
1007 | 1007 |
| |
1008 | 1008 |
| |
| 1009 | + | |
| 1010 | + | |
1009 | 1011 |
| |
1010 | 1012 |
| |
1011 | 1013 |
| |
|
0 commit comments
Comments
(0)