Re: Show WAL write and fsync stats in pg_stat_io

From: Nazir Bilal Yavuz <byavuz81(at)gmail(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Bertrand Drouvot <bertranddrouvot(dot)pg(at)gmail(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Melanie Plageman <melanieplageman(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, "bharath(dot)rupireddyforpostgres(at)gmail(dot)com" <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, Thomas Munro <thomas(dot)munro(at)gmail(dot)com>
Subject: Re: Show WAL write and fsync stats in pg_stat_io
Date: 2025-02-06 12:39:05
Message-ID: CAN55FZ09UhUY8Mwvxi3uB7bC0hM3gPQ9vrZEaHy+un+hFs+Zng@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On Thu, 6 Feb 2025 at 11:39, Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>
> On Thu, Feb 06, 2025 at 07:32:01AM +0000, Bertrand Drouvot wrote:
> > That sounds ok, but I wonder if that's the best appropriate place. I could
> > think of the checkpointer test in 029_stats_restart.pl and the startup/standby
> > one in one related to standby (030_stats_cleanup_replica.pl?). Though that's
> > probably just a matter of taste.
>
> Hmm. Your suggestion of 029_stats_restart.pl is a tempting choice,
> indeed, more in line with the fact that we are checking some stats
> data. I did not think about that.
>
> Note that I'm OK about using 029 or even 030, as long as both queries
> stay together.

My vote goes to 029. It already has the '## check checkpoint and wal
stats are incremented due to restart' part and what we are adding is
similar to that.

On Thu, 6 Feb 2025 at 10:32, Bertrand Drouvot
<bertranddrouvot(dot)pg(at)gmail(dot)com> wrote:
>
> s/and the primary WAL some writes/and the primary some WAL writes/

I am not a native speaker but 'primary writes some WAL with its
checkpointer' sounds better to me.

--
Regards,
Nazir Bilal Yavuz
Microsoft

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jelte Fennema-Nio 2025-02-06 12:53:03 Re: new commitfest transition guidance
Previous Message Rushabh Lathia 2025-02-06 12:38:46 Support NOT VALID / VALIDATE constraint options for named NOT NULL constraints