From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | David Rowley <dgrowleyml(at)gmail(dot)com> |
Cc: | Sayyid Ali Sajjad Rizavi <sasrizavi(at)gmail(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: Allow round() function to accept float and double precision |
Date: | 2022-12-01 01:45:38 |
Message-ID: | 1480693.1669859138@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
David Rowley <dgrowleyml(at)gmail(dot)com> writes:
> We do have some weirdness in some existing overloaded functions.
> pg_size_pretty() is an example.
> If you run: SELECT pg_size_pretty(1000); you get:
> ERROR: function pg_size_pretty(integer) is not unique
Yeah, you have to be careful about that when proposing to overload
a function name.
> That occurs because we don't know if we should promote the INT into a
> BIGINT or into a NUMERIC. We have a pg_size_pretty() function for each
> of those. I don't think the same polymorphic type resolution problem
> exists for REAL, FLOAT8 and NUMERIC.
I would counsel against bothering with a REAL version. FLOAT8 will
cover that case just fine.
> I'm unsure what the repercussions of the fact that REAL and FLOAT8 are
> not represented as decimals.
The main thing is that I think the output will still have to be
NUMERIC, or you're going to get complaints about "inaccurate"
results. Before we got around to inventing infinities for NUMERIC,
that choice would have been problematic, but now I think it's OK.
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2022-12-01 02:05:54 | Re: Add LZ4 compression in pg_dump |
Previous Message | David Rowley | 2022-12-01 01:30:44 | Re: Allow round() function to accept float and double precision |