On Mon, Jan 3, 2022 at 10:20 AM Tom Lane <t...@sss.pgh.pa.us> wrote:
> BTW, if you want to get rid of the quotes, I think that something
> else has to be done to set off the type name from the rest. In
> this instance someone might think that we're complaining about a
> "time zone unit", whatever that is. I suggest swapping it around to
>
> units \"%s\" not recognized for type %s
>
> Also, personally, I'd write unit not units, but that's
> more debatable.
Your suggestion sounds good to me. I'll update the patch with that.
Not that it changes anything, I think, but the wording ambiguity you
mention is present today in the timestamptz error message:
benesch=> select extract('nope' from now());
ERROR: timestamp with time zone units "nope" not recognized
