Hi,
On Thu, 6 Feb 2025 at 11:39, Michael Paquier <michael@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.pg@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
