Re: pgsql: Only allow returning string types or bytea from json_serialize

From: Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>
To: andrew(at)dunslane(dot)net
Cc: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: Re: pgsql: Only allow returning string types or bytea from json_serialize
Date: 2022-07-08 06:41:35
Message-ID: 20220708.154135.2123613118233840495.horikyota.ntt@gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-committers

At Thu, 07 Jul 2022 21:46:12 +0000, Andrew Dunstan <andrew(at)dunslane(dot)net> wrote in
> Only allow returning string types or bytea from json_serialize

I noticed that this introcues the following error message.

+ errmsg("cannot use RETURNING type %s in JSON_SERIALIZE"
+ format_type_be(returning->typid)),

However, the same file has the following error message.

> errmsg("cannot use RETURNING type %s in %s",
> format_type_be(returning->typid), fname),

So, couldn't we share the format string to reduce translatable
messages?

And, the other messages are

cannot use RETURNING type %s in JSON_SCALAR(), and
cannot use RETURNING type %s in JSON()

So, I think this should not be

cannot use RETURNING type %s in JSON_SERIALIZE

, but should be

cannot use RETURNING type %s in JSON_SERIALIZE()

regards.

--
Kyotaro Horiguchi
NTT Open Source Software Center

Attachment Content-Type Size
0001-Fix-an-error-message.patch text/x-patch 1.9 KB

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Peter Eisentraut 2022-07-08 07:32:12 pgsql: Reformat some more node comments
Previous Message Thomas Munro 2022-07-08 02:18:20 pgsql: Remove HP/Intel Itanium support.