Re: Printing LSN made easy

From: Craig Ringer <craig(dot)ringer(at)enterprisedb(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Michael Paquier <michael(at)paquier(dot)xyz>, Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>
Subject: Re: Printing LSN made easy
Date: 2020-11-30 02:12:35
Message-ID: CAGRY4nxGKTCM0k-hw0vyE24c2e+si5FB0hOWaC_fwg2rdvTTyg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, Nov 30, 2020 at 1:10 AM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:

> Michael Paquier <michael(at)paquier(dot)xyz> writes:
> > On Fri, Nov 27, 2020 at 04:10:27PM +0530, Ashutosh Bapat wrote:
> >> Off list Craig Ringer suggested introducing a new format specifier
> >> similar to %m for LSN but I did not get time to take a look at the
> >> relevant code. AFAIU it's available only to elog/ereport, so may not
> >> be useful generally. But teaching printf variants about the new format
> >> would be the best solution. However, I didn't find any way to do that.
>
> > -1. %m maps to errno, that is much more generic. A set of macros
> > that maps to our internal format would be fine enough IMO.
>
> Agreed. snprintf.c is meant to implement a recognized standard
> (ok, %m is a GNU extension, but it's still pretty standard).
> I'm not on board with putting PG-only extensions in there.
>

Fair enough. I did not realise that %m was a GNU extension (never looked
closely) so I thought we had precedent for Pg specific extensions there.

Agreed in that case, no argument from me.

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2020-11-30 02:18:15 Re: [sqlsmith] Planner error on lateral joins
Previous Message Fujii Masao 2020-11-30 02:12:13 Re: Use standard SIGHUP and SIGTERM handlers in autoprewarm module