From: | Joe Conway <mail(at)joeconway(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | pgsql-hackers(at)lists(dot)postgresql(dot)org |
Subject: | Re: Improving contrib/tablefunc's error reporting |
Date: | 2024-03-09 22:34:41 |
Message-ID: | 5d315efd-18d3-4874-acfb-3d663ae37017@joeconway.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 3/9/24 15:39, Tom Lane wrote:
> Joe Conway <mail(at)joeconway(dot)com> writes:
>> On 3/9/24 13:07, Tom Lane wrote:
>>>> BTW, while I didn't touch it here, it seems fairly bogus that
>>>> connectby() checks both type OID and typmod for its output
>>>> columns while crosstab() only checks type OID. I think
>>>> crosstab() is in the wrong and needs to be checking typmod.
>>>> That might be fit material for a separate patch though.
>
>> Something like the attached what you had in mind? (applies on top of
>> your two patches)
>
> Yeah, exactly.
>
> As far as the comment change goes:
>
> - * - attribute [1] of the sql tuple is the category; no need to check it -
> - * attribute [2] of the sql tuple should match attributes [1] to [natts]
> + * attribute [1] of the sql tuple is the category; no need to check it
> + * attribute [2] of the sql tuple should match attributes [1] to [natts - 1]
> * of the return tuple
>
> I suspect that this block looked better when originally committed but
> didn't survive contact with pgindent. You should check whether your
> version will; if not, some dashes on the /* line will help.
Thanks for the review and heads up. I fiddled with it a bit to make it
pgindent clean. I saw you commit your patches so I just pushed mine.
--
Joe Conway
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com
From | Date | Subject | |
---|---|---|---|
Next Message | Andy Fan | 2024-03-09 23:12:27 | Re: remaining sql/json patches |
Previous Message | Melanie Plageman | 2024-03-09 20:41:49 | Re: Combine Prune and Freeze records emitted by vacuum |