Re: Improving contrib/tablefunc's error reporting

2024-03-09 Thread Joe Conway
On 3/9/24 15:39, Tom Lane wrote: Joe Conway 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

Re: Improving contrib/tablefunc's error reporting

2024-03-09 Thread Tom Lane
Joe Conway 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

Re: Improving contrib/tablefunc's error reporting

2024-03-09 Thread Joe Conway
On 3/9/24 13:07, Tom Lane wrote: Joe Conway writes: On 3/5/24 17:04, Tom Lane wrote: After reading the thread at [1], I could not escape the feeling that contrib/tablefunc's error reporting is very confusing. The changes all look good to me and indeed more consistent with the docs. Do you

Re: Improving contrib/tablefunc's error reporting

2024-03-09 Thread Tom Lane
Joe Conway writes: > On 3/5/24 17:04, Tom Lane wrote: >> After reading the thread at [1], I could not escape the feeling >> that contrib/tablefunc's error reporting is very confusing. > The changes all look good to me and indeed more consistent with the docs. > Do you want me to push these? If

Re: Improving contrib/tablefunc's error reporting

2024-03-09 Thread Joe Conway
On 3/5/24 17:04, Tom Lane wrote: After reading the thread at [1], I could not escape the feeling that contrib/tablefunc's error reporting is very confusing. Looking into the source code, I soon found that it is also very inconsistent, with similar error reports being phrased quite differently.

Re: Improving contrib/tablefunc's error reporting

2024-03-05 Thread Tom Lane
Joe Conway writes: > I will have a look at your patches, and the other issue you mention, but > it might be a day or three before I can give it some quality time. No hurry certainly. Thanks for looking. regards, tom lane

Re: Improving contrib/tablefunc's error reporting

2024-03-05 Thread Joe Conway
On 3/5/24 17:04, Tom Lane wrote: After reading the thread at [1], I could not escape the feeling that contrib/tablefunc's error reporting is very confusing. Looking into the source code, I soon found that it is also very inconsistent, with similar error reports being phrased quite differently.

Improving contrib/tablefunc's error reporting

2024-03-05 Thread Tom Lane
After reading the thread at [1], I could not escape the feeling that contrib/tablefunc's error reporting is very confusing. Looking into the source code, I soon found that it is also very inconsistent, with similar error reports being phrased quite differently. The terminology for column names