forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commitf90149e
committed
Don't use custom OID symbols in pg_type.dat, either.
On the same reasoning as in commit36b9312, forbid using customoid_symbol macros in pg_type as well as pg_proc, so that we alwaysrely on the predictable macro names generated by genbki.pl.We do continue to grant grandfather status to the names CASHOID andLSNOID, although those are now considered deprecated aliases for thepreferred names MONEYOID and PG_LSNOID. This is because there'slikely to be client-side code using the old names, and this bout ofneatnik-ism doesn't quite seem worth breaking client code.There might be a case for grandfathering EVTTRIGGEROID, too, sinceexternally-maintained PLs may reference that symbol. But renamingsuch references to EVENT_TRIGGEROID doesn't seem like a particularlyheavy lift --- we make far more significant backend API changes inevery major release. For now I didn't add that, but we couldreconsider if there's pushback.The other names changed here seem pretty unlikely to have any outsideuses. Again, we could add alias macros if there are complaints, butfor now I didn't.As before, no need for a catversion bump.John NaylorDiscussion:https://postgr.es/m/CAFBsxsHpCbjfoddNGpnnnY5pHwckWfiYkMYSF74PmP1su0+ZOw@mail.gmail.com1 parent1c7675a commitf90149e
File tree
13 files changed
+38
-37
lines changed- src
- backend
- bootstrap
- catalog
- commands
- utils/misc
- fe_utils
- include/catalog
- interfaces/ecpg/ecpglib
- pl
- plperl
- plpgsql/src
- plpython
- tcl
13 files changed
+38
-37
lines changedLines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
133 | 133 |
| |
134 | 134 |
| |
135 | 135 |
| |
136 |
| - | |
| 136 | + | |
137 | 137 |
| |
138 | 138 |
| |
139 | 139 |
| |
|
Lines changed: 6 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
588 | 588 |
| |
589 | 589 |
| |
590 | 590 |
| |
591 |
| - | |
592 |
| - | |
| 591 | + | |
593 | 592 |
| |
| 593 | + | |
| 594 | + | |
| 595 | + | |
| 596 | + | |
| 597 | + | |
594 | 598 |
| |
595 | 599 |
| |
596 | 600 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
177 | 177 |
| |
178 | 178 |
| |
179 | 179 |
| |
180 |
| - | |
| 180 | + | |
181 | 181 |
| |
182 | 182 |
| |
183 | 183 |
| |
|
Lines changed: 5 additions & 5 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
94 | 94 |
| |
95 | 95 |
| |
96 | 96 |
| |
97 |
| - | |
| 97 | + | |
98 | 98 |
| |
99 |
| - | |
| 99 | + | |
100 | 100 |
| |
101 | 101 |
| |
102 | 102 |
| |
| |||
223 | 223 |
| |
224 | 224 |
| |
225 | 225 |
| |
226 |
| - | |
| 226 | + | |
227 | 227 |
| |
228 | 228 |
| |
229 | 229 |
| |
230 |
| - | |
| 230 | + | |
231 | 231 |
| |
232 |
| - | |
| 232 | + | |
233 | 233 |
| |
234 | 234 |
| |
235 | 235 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
3495 | 3495 |
| |
3496 | 3496 |
| |
3497 | 3497 |
| |
3498 |
| - | |
| 3498 | + | |
3499 | 3499 |
| |
3500 | 3500 |
| |
3501 | 3501 |
| |
|
Lines changed: 10 additions & 20 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
15 | 15 |
| |
16 | 16 |
| |
17 | 17 |
| |
18 |
| - | |
19 |
| - | |
20 |
| - | |
| 18 | + | |
| 19 | + | |
21 | 20 |
| |
22 | 21 |
| |
23 |
| - | |
24 |
| - | |
25 |
| - | |
26 | 22 |
| |
27 | 23 |
| |
28 | 24 |
| |
| |||
144 | 140 |
| |
145 | 141 |
| |
146 | 142 |
| |
147 |
| - | |
148 |
| - | |
| 143 | + | |
149 | 144 |
| |
150 | 145 |
| |
151 | 146 |
| |
152 | 147 |
| |
153 | 148 |
| |
154 |
| - | |
155 |
| - | |
| 149 | + | |
156 | 150 |
| |
157 | 151 |
| |
158 | 152 |
| |
159 | 153 |
| |
160 | 154 |
| |
161 |
| - | |
162 |
| - | |
| 155 | + | |
163 | 156 |
| |
164 | 157 |
| |
165 | 158 |
| |
166 | 159 |
| |
167 | 160 |
| |
168 |
| - | |
169 |
| - | |
| 161 | + | |
170 | 162 |
| |
171 | 163 |
| |
172 | 164 |
| |
173 | 165 |
| |
174 |
| - | |
175 |
| - | |
| 166 | + | |
176 | 167 |
| |
177 | 168 |
| |
178 | 169 |
| |
| |||
237 | 228 |
| |
238 | 229 |
| |
239 | 230 |
| |
240 |
| - | |
| 231 | + | |
241 | 232 |
| |
242 | 233 |
| |
243 | 234 |
| |
| |||
409 | 400 |
| |
410 | 401 |
| |
411 | 402 |
| |
412 |
| - | |
413 |
| - | |
| 403 | + | |
414 | 404 |
| |
415 | 405 |
| |
416 | 406 |
| |
| |||
542 | 532 |
| |
543 | 533 |
| |
544 | 534 |
| |
545 |
| - | |
| 535 | + | |
546 | 536 |
| |
547 | 537 |
| |
548 | 538 |
| |
|
Lines changed: 7 additions & 0 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
312 | 312 |
| |
313 | 313 |
| |
314 | 314 |
| |
| 315 | + | |
| 316 | + | |
| 317 | + | |
| 318 | + | |
| 319 | + | |
| 320 | + | |
| 321 | + | |
315 | 322 |
| |
316 | 323 |
| |
317 | 324 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
230 | 230 |
| |
231 | 231 |
| |
232 | 232 |
| |
233 |
| - | |
| 233 | + | |
234 | 234 |
| |
235 | 235 |
| |
236 | 236 |
| |
|
Lines changed: 2 additions & 2 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2002 | 2002 |
| |
2003 | 2003 |
| |
2004 | 2004 |
| |
2005 |
| - | |
| 2005 | + | |
2006 | 2006 |
| |
2007 | 2007 |
| |
2008 | 2008 |
| |
| |||
2838 | 2838 |
| |
2839 | 2839 |
| |
2840 | 2840 |
| |
2841 |
| - | |
| 2841 | + | |
2842 | 2842 |
| |
2843 | 2843 |
| |
2844 | 2844 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
551 | 551 |
| |
552 | 552 |
| |
553 | 553 |
| |
554 |
| - | |
| 554 | + | |
555 | 555 |
| |
556 | 556 |
| |
557 | 557 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
445 | 445 |
| |
446 | 446 |
| |
447 | 447 |
| |
448 |
| - | |
| 448 | + | |
449 | 449 |
| |
450 | 450 |
| |
451 | 451 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
220 | 220 |
| |
221 | 221 |
| |
222 | 222 |
| |
223 |
| - | |
| 223 | + | |
224 | 224 |
| |
225 | 225 |
| |
226 | 226 |
| |
|
Lines changed: 1 addition & 1 deletion
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
1532 | 1532 |
| |
1533 | 1533 |
| |
1534 | 1534 |
| |
1535 |
| - | |
| 1535 | + | |
1536 | 1536 |
| |
1537 | 1537 |
| |
1538 | 1538 |
| |
|
0 commit comments
Comments
(0)