Re: default result formats setting

From: David Steele <david(at)pgmasters(dot)net>
To: Andrew Dunstan <andrew(at)dunslane(dot)net>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Subject: Re: default result formats setting
Date: 2021-03-09 14:47:28
Message-ID: bb359123-039e-d114-3b5c-8188197b6f73@pgmasters.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 11/25/20 2:06 AM, Peter Eisentraut wrote:
> On 2020-11-16 16:15, Andrew Dunstan wrote:
>> I think this is conceptually OK, although it feels a bit odd.
>>
>> Might it be better to have the values as typename={binary,text} pairs
>> instead of oid={0,1} pairs, which are fairly opaque? That might make
>> things easier for things like UDTs where the oid might not be known or
>> constant.
>
> Yes, type names would be better.  I was hesitant because of all the
> parsing work involved, but I bit the bullet and did it in the new patch.
>
> To simplify the format, I changed the parameter so it's just a list of
> types that you want in binary, rather than type=value pairs.  If we ever
> want to add another format, we would revisit this, but it seems unlikely
> in the near future.
>
> Also, I have changed the naming of the parameter since this is no longer
> the "default" but something you choose explicitly.  I'm thinking in the
> direction of "auto" mode for the naming.  Obviously, the name is easy to
> tweak in any case.

Andrew, Tom, does the latest patch address your concerns?

Regards,
--
-David
david(at)pgmasters(dot)net

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Eisentraut 2021-03-09 14:51:41 Re: authtype parameter in libpq
Previous Message David Steele 2021-03-09 14:28:45 Re: Any objections to implementing LogicalDecodeMessageCB for pgoutput?