On 7/26/21, 5:48 PM, "Andres Freund" <and...@anarazel.de> wrote: > On 2021-07-26 20:27:21 +0000, Bossart, Nathan wrote: >> +1. I was confused by this when working on a WAL pre-allocation >> patch [0]. Perhaps it could be replaced by a new parameter and a new >> field in pg_stat_wal. How about something like log_wal_init_interval, >> where the value is the minimum amount of time between reporting the >> number of WAL segments created since the last report? > > Why not just make the number in log_checkpoints accurate? There's no > point in the current number, so we don't need to preserve it...
My understanding is that the statistics logged for log_checkpoints are just for that specific checkpoint. From that angle, the value for the number of WAL files added is technically correct. Checkpoints will only ever create zero or one new files via PreallocXlogFiles(). If we also added all the segments created outside of the checkpoint, the value for "added" would go from meaning "WAL files created by this checkpoint" to "WAL files creates since the last checkpoint." That's probably less confusing than what's there today, but it's still slightly different than all the other log_checkpoints stats. Nathan
