- Notifications
You must be signed in to change notification settings - Fork4.9k
Commitb2e09fc
committed
This is not the only place in the system catalogs where NULL is
effectively used to mean a default value that could also be spelledout explicitly. (ACLs behave that way, and useconfig/datconfigdo too IIRC.)It's a bit of a hack, but it saves table space and backend code ---without this convention the default would have to be inserted "manually"since we have no mechanism to supply defaults when C code is forming anew catalog tuple.I'm inclined to leave the code alone. But Alvaro is right that it'd begood to point out the 'infinity' option in the CREATE USER and ALTERUSER man pages. (Doc patch please?)Alvaro Herrera1 parent0fe77d7 commitb2e09fc
1 file changed
+10
-2
lines changedLines changed: 10 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1 | 1 |
| |
2 |
| - | |
| 2 | + | |
3 | 3 |
| |
4 | 4 |
| |
5 | 5 |
| |
| |||
124 | 124 |
| |
125 | 125 |
| |
126 | 126 |
| |
127 |
| - | |
| 127 | + | |
| 128 | + | |
128 | 129 |
| |
129 | 130 |
| |
130 | 131 |
| |
| |||
232 | 233 |
| |
233 | 234 |
| |
234 | 235 |
| |
| 236 | + | |
| 237 | + | |
| 238 | + | |
| 239 | + | |
| 240 | + | |
| 241 | + | |
| 242 | + | |
235 | 243 |
| |
236 | 244 |
| |
237 | 245 |
| |
|
0 commit comments
Comments
(0)