Hi,

On Thu, 6 Feb 2025 at 11:39, Michael Paquier <mich...@paquier.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...@gmail.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


Reply via email to