forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commit879808e
committed
Avoid promising that "ADD COLUMN ... DEFAULT NULL" is free.
The system realizes that DEFAULT NULL is dummy in simple cases, but not ifa cast function (such as a length coercion) needs to be applied. It'sdubious that suppressing that function call would be appropriate, anyway.For the moment, let's just adjust the docs to say that you should omit theDEFAULT clause if you don't want a rewrite to happen. Per gripe from AmitLangote.1 parent04e298b commit879808e
1 file changed
+8
-4
lines changedLines changed: 8 additions & 4 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
854 | 854 |
| |
855 | 855 |
| |
856 | 856 |
| |
| 857 | + | |
| 858 | + | |
| 859 | + | |
857 | 860 |
| |
858 | 861 |
| |
859 | 862 |
| |
860 |
| - | |
861 |
| - | |
862 |
| - | |
| 863 | + | |
| 864 | + | |
| 865 | + | |
| 866 | + | |
863 | 867 |
| |
864 |
| - | |
| 868 | + | |
865 | 869 |
| |
866 | 870 |
| |
867 | 871 |
| |
|
0 commit comments
Comments
(0)