Re: A modest proposal: let's add PID to assertion failure messages - Mailing list pgsql-hackers

From Michael Paquier
Subject Re: A modest proposal: let's add PID to assertion failure messages
Date
Msg-id 20201005005824.GG1464@paquier.xyz
Whole thread Raw
In response to Re: A modest proposal: let's add PID to assertion failure messages  (Thomas Munro <thomas.munro@gmail.com>)
Responses Re: A modest proposal: let's add PID to assertion failure messages
List pgsql-hackers
On Mon, Oct 05, 2020 at 10:20:01AM +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.  (log_line_prefix includes %p in its default configuration for the
TAP tests).
--
Michael

Attachment

pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Buggy handling of redundant options in COPY
Next
From: Michael Paquier
Date:
Subject: Re: Add header support to text format and matching feature