Re: add checkpoint stats of snapshot and mapping files of pg_logical dir
От | Nathan Bossart |
---|---|
Тема | Re: add checkpoint stats of snapshot and mapping files of pg_logical dir |
Дата | |
Msg-id | 20220321202448.GA1513050@nathanxps13 обсуждение исходный текст |
Ответ на | Re: add checkpoint stats of snapshot and mapping files of pg_logical dir (Bharath Rupireddy <bharath.rupireddyforpostgres@gmail.com>) |
Список | pgsql-hackers |
On Mon, Mar 21, 2022 at 11:27:15AM +0530, Bharath Rupireddy wrote: > On Sat, Mar 19, 2022 at 3:16 AM Nathan Bossart <nathandbossart@gmail.com> wrote: >> /* buffer stats */ >> appendStringInfo(&logmsg, "wrote %d buffers (%.1f%%); ", >> CheckpointStats.ckpt_bufs_written, >> (double) CheckpointStats.ckpt_bufs_written * 100 / NBuffers); >> >> /* WAL file stats */ >> appendStringInfo(&logmsg, "%d WAL file(s) added, %d removed, %d recycled; ", >> CheckpointStats.ckpt_segs_added, >> CheckpointStats.ckpt_segs_removed, >> CheckpointStats.ckpt_segs_recycled); > > Do we actually need to granularize the message like this? I actually > don't think so, because we print the stats even if they are 0 (buffers > written is 0 or WAL segments added is 0 and so on). I suggested it because I thought it would improve readability and simplify optionally adding new stats to the message. If there is another way to accomplish those things, that is fine by me. -- Nathan Bossart Amazon Web Services: https://aws.amazon.com
В списке pgsql-hackers по дате отправления: