| From: | Andres Freund <andres(at)anarazel(dot)de> |
|---|---|
| To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
| Cc: | Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Frontend error logging style |
| Date: | 2022-03-22 02:13:48 |
| Message-ID: | 20220322021348.pvtvgrseyfohkxob@alap3.anarazel.de |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi,
On 2022-03-21 22:00:37 -0400, Tom Lane wrote:
> Andres Freund <andres(at)anarazel(dot)de> writes:
> > This unfortunately has had some bitrot: http://cfbot.cputube.org/patch_37_3574.log
>
> Yeah, I know. That patch touches enough places that it's sure to
> break every few days during a commitfest. I'm not real excited about
> maintaining it reactively.
Makes sense. I'd leave it on waiting-on-author then, so that reviewers looking
through the CF don't need to look at it?
> Maybe the plan could be to push it at the end of the commitfest?
Would make sense to me.
Arguably parts of it could be committed sooner than that, e.g. exit(-1). But
that'd not make it meaningfully easier to maintain, so ...
> But what I'd really like at the moment is buy-in or rejection of the whole
> concept, so I know whether it's worth spending more time on.
I'm +1. I've not carefully looked through every single changed callsite, but
IMO it looks like a clear improvement.
Greetings,
Andres Freund
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Thomas Munro | 2022-03-22 02:16:06 | Re: Why is src/test/modules/committs/t/002_standby.pl flaky? |
| Previous Message | Tom Lane | 2022-03-22 02:06:46 | Re: pgsql: Add ALTER SUBSCRIPTION ... SKIP. |