Re: BUG #18240: Undefined behaviour in cash_mul_flt8() and friends
Tom Lane <tgl@sss.pgh.pa.us>
From: Tom Lane <tgl@sss.pgh.pa.us>
To: David Rowley <dgrowleyml@gmail.com>
Cc: Michael Paquier <michael@paquier.xyz>, exclusion@gmail.com,
pgsql-bugs@lists.postgresql.org
Date: 2023-12-12T14:37:58Z
Lists: pgsql-bugs
David Rowley <dgrowleyml@gmail.com> writes: > On Tue, 12 Dec 2023 at 22:03, Michael Paquier <michael@paquier.xyz> wrote: >> And numeric has much better code coverage and support. I am wondering >> whether we've reached the point where it would be better to remove it >> entirely from the tree, and just tell people to use numeric. > We could extract it into a contrib module. Perhaps, but ... > That might help reduce new usages of it and would also allow people > who have large tables using the money type but can't realistically > wait out a table rewrite to upgrade to a newer version of Postgres. ... I doubt that a contrib module would solve the problem for people who can't afford a rewrite. pg_upgrade requires that datatype OIDs stay the same, which is something I don't believe a contrib module could manage. We've run into that before if memory serves. Is it time to do something about that? Perhaps we could allow extension modules to use binary_upgrade_set_next_pg_type_oid, and then somehow reserve the money and _money OIDs forever? regards, tom lane
Commits
-
Add overflow checks to money type.
- e8dfe0430fad 14.13 landed
- c5321e9651c5 13.16 landed
- b82791c8fc2c 15.8 landed
- 4f962815871f 12.20 landed
- 3764ee47f78c 17.0 landed
- 34e9dce69c46 16.4 landed
- 22b0ccd65d27 18.0 landed
-
Reduce dependency to money data type in main regression test suite
- 31acee4b66f9 17.0 landed