| From: | Craig Ringer <craig(at)2ndquadrant(dot)com> |
|---|---|
| To: | Andres Freund <andres(at)anarazel(dot)de> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Julien Rouhaud <rjuju123(at)gmail(dot)com>, Masahiko Sawada <masahiko(dot)sawada(at)2ndquadrant(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Missing errcode() in ereport |
| Date: | 2020-03-30 03:54:03 |
| Message-ID: | CAMsr+YFypZJXfT18OrXgZzc0XULw1Eppjwg0sizuBz5Lnb3KJw@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Fri, 20 Mar 2020, 01:59 Andres Freund, <andres(at)anarazel(dot)de> wrote:
> Hi,
>
> On 2020-03-17 10:09:18 -0400, Tom Lane wrote:
> > We might want to spend some effort thinking how to find or prevent
> > additional bugs of the same ilk ...
>
> Yea, that'd be good. Trying to help people new to postgres write their
> first patches I found that ereport is very confusing to them - largely
> because the syntax doesn't make much sense. Made worse by the compiler
> error messages being terrible in many cases.
Very much agreed.
I'd have found it helpful to just have the docs explain clearly how it
works by chaining the comma operator using functions with ignored return
values.
That would also help people understand how they can make parts of an
ereport conditional, e.g. only set errdetail() if there additional info is
currently available W/O duplicating the rest of the ereport .
Not sure there's much we can do without changing ereport's "signature"
> though :(
>
> Regards,
>
> Andres
>
>
>
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Tom Lane | 2020-03-30 03:57:50 | Re: Missing errcode() in ereport |
| Previous Message | Craig Ringer | 2020-03-30 03:50:23 | Re: [PATCH] Support built control file in PGXS VPATH builds |