pg_stat_wal: Accumulate time as instr_time instead of microseconds
authorAndres Freund <andres@anarazel.de>
Thu, 30 Mar 2023 21:23:14 +0000 (14:23 -0700)
committerAndres Freund <andres@anarazel.de>
Thu, 30 Mar 2023 21:23:14 +0000 (14:23 -0700)
commitca7b3c4c00042038ba9c282c4807e05c0a527e42
tree73ede8f1a9988dc4f699d2b2c95c75a7982af531
parent122376f028a0e31b91d6c6bad2a9a6e994708547
pg_stat_wal: Accumulate time as instr_time instead of microseconds

In instr_time.h it is stated that:

* When summing multiple measurements, it's recommended to leave the
* running sum in instr_time form (ie, use INSTR_TIME_ADD or
* INSTR_TIME_ACCUM_DIFF) and convert to a result format only at the end.

The reason for that is that converting to microseconds is not cheap, and can
loose precision.  Therefore this commit changes 'PendingWalStats' to use
'instr_time' instead of 'PgStat_Counter' while accumulating 'wal_write_time'
and 'wal_sync_time'.

Author: Nazir Bilal Yavuz <byavuz81@gmail.com>
Reviewed-by: Andres Freund <andres@anarazel.de>
Reviewed-by: Kyotaro Horiguchi <horikyota.ntt@gmail.com>
Reviewed-by: Melanie Plageman <melanieplageman@gmail.com>
Discussion: https://postgr.es/m/1feedb83-7aa9-cb4b-5086-598349d3f555@gmail.com
src/backend/access/transam/xlog.c
src/backend/storage/file/buffile.c
src/backend/utils/activity/pgstat_wal.c
src/include/pgstat.h
src/tools/pgindent/typedefs.list