From: Nazir Bilal Yavuz <byavu...@gmail.com> Sent: Thursday, June 27, 2024 9:01 PM > > Hi, > > On Thu, 27 Jun 2024 at 14:30, Melanie Plageman > <melanieplage...@gmail.com> wrote: > > > > On Thu, Jun 27, 2024 at 5:06 AM <hajime.matsun...@nttdata.com> wrote: > > > > > > Hi, > > > > > > pg_stat_io has I/O statistics that are collected when track_io_timing is > > > enabled, but it is not mentioned in the description of track_io_timing. > > > I think it's better to add a description of pg_stat_io for easy reference. > > > What do you think? > > > > Seems quite reasonable to me given that track_wal_io_timing mentions > > pg_stat_wal. I noticed that the sentence about track_io_timing in the > > statistics collection configuration section [1] only mentions reads > > and writes -- perhaps it should also mention extends and fsyncs? > > Both suggestions look good to me. If what you said will be > implemented, maybe track_wal_io_timing too should mention fsyncs?
Thanks for the suggestions the other day. I have created a patch that incorporates your suggestions. Regards, -- Hajime Matsunaga NTT DATA Group Corporation
v2-0001-Doc-fix-track_io_timing-description-to-mention-pg.patch
Description: v2-0001-Doc-fix-track_io_timing-description-to-mention-pg.patch