Re: BUG #17866: behavior does not match documentation

Daniel Gustafsson <daniel@yesql.se>

From: Daniel Gustafsson <daniel@yesql.se>
To: Tom Lane <tgl@sss.pgh.pa.us>
Cc: eugeny@zhuzhnev.com, pgsql-bugs <pgsql-bugs@lists.postgresql.org>, Peter Eisentraut <peter.eisentraut@enterprisedb.com>
Date: 2023-03-23T22:00:24Z
Lists: pgsql-bugs
> On 23 Mar 2023, at 22:54, Tom Lane <tgl@sss.pgh.pa.us> wrote:
> 
> Daniel Gustafsson <daniel@yesql.se> writes:
>> On 23 Mar 2023, at 19:31, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>>> I wonder if we should rethink that and have these operations strip
>>> insignificant trailing zeroes.
> 
>> Any app relying on insignificant trailing zeroes seems broken, but the inverse
>> can be argued as well.  It's however quite easy to argue for the stripped
>> output being more readable though.
> 
> It's more readable for sure, and it duplicates what you got in pre-v14
> versions, at least textually.  I'm not sure I'd propose back-patching
> this, but it feels like a good idea for HEAD.

Agreed, I don't think this is material for backpatching.

--
Daniel Gustafsson




Commits

  1. Doc: adjust examples of EXTRACT() output to match current reality.

  2. Change return type of EXTRACT to numeric