Re: pg_stat_wal_receiver and flushedUpto/writtenUpto - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: pg_stat_wal_receiver and flushedUpto/writtenUpto
Date
Msg-id 20200515233446.GA334@alvherre.pgsql
Whole thread Raw
In response to Re: pg_stat_wal_receiver and flushedUpto/writtenUpto  (Michael Paquier <michael@paquier.xyz>)
Responses Re: pg_stat_wal_receiver and flushedUpto/writtenUpto  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
On 2020-May-16, Michael Paquier wrote:

> On Fri, May 15, 2020 at 01:43:11PM -0400, Alvaro Herrera wrote:
> > Why do you put the column at the end?  I would put written_lsn before
> > flushed_lsn.
> 
> Fine by me.  I was thinking yesterday about putting the written
> position after the flushed one, and finished with something that maps
> with the structure.

IIRC the only reason to put the written LSN where it is is so that it's
below the mutex, to indicate it's not protected by it.  Conceptually,
the written LSN is "before" the flushed LSN, which is why I propose to
put it ahead of it.

> > Since this requires a catversion bump, I think it'd be best to do it
> > before beta1 next week.
> 
> Yes.  What do you think of the attached?

Yeah, that seems good (I didn't verify the boilerplate in
pg_stat_get_wal_receiver or pg_proc.dat).  I propose

+       Last write-ahead log location already received and written to
+       disk, but not flushed.  This should not be used for data
+       integrity checks.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services



pgsql-hackers by date:

Previous
From: Alvaro Herrera
Date:
Subject: Re: pg13: xlogreader API adjust
Next
From: Alvaro Herrera
Date:
Subject: Re: Potentially misleading name of libpq pass phrase hook