Re: A modest proposal: let's add PID to assertion failure messages
Andres Freund <andres@anarazel.de>
From: Andres Freund <andres@anarazel.de>
To: Thomas Munro <thomas.munro@gmail.com>
Cc: Tom Lane <tgl@sss.pgh.pa.us>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2020-10-06T18:27:48Z
Lists: pgsql-hackers
On 2020-10-05 10:20:01 +1300, Thomas Munro wrote: > On Mon, Oct 5, 2020 at 10:08 AM Tom Lane <tgl@sss.pgh.pa.us> wrote: > > In these days when we run almost all test cases in parallel, it's > > frequently not that easy to tie a "TRAP: ..." message in the log > > to nearby log messages. (The postmaster's subsequent complaint > > often helps, but it could be some distance away in the log; and > > good luck untangling things if more than one Assert failure happens > > concurrently.) We could add a simple bread crumb trail by > > including the process's PID in such messages. Any objections? > > +1 +1
Commits
-
Include the process PID in assertion-failure messages.
- 18c170a08ee2 14.0 landed