pgstat: reduce timer overhead by leaving timer running.
Previously the timer was enabled whenever there were any pending stats after
executing a statement, just to then be disabled again when not idle
anymore. That lead to an increase in GetCurrentTimestamp() calls from within
timeout.c compared to 14.
To avoid that increase, leave the timer enabled until stats are reported,
rather than until idle. The timer is only disabled once the pending stats have
been reported.
For me this fixes the increase in GetCurrentTimestamp() calls, there now are
fewer calls in 15 than in 14, in the previously slowed down workload.
While at it, also update assertion in pgstat_report_stat() to be more precise.
Author: Andres Freund <andres@anarazel.de>
Discussion: https://postgr.es/m/20220616233130.rparivafipt6doj3@alap3.anarazel.de
Backpatch: 15-
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/056cc366fafa8110f5761a7383ae1cd903dd728c
Modified Files
--------------
src/backend/tcop/postgres.c | 48 ++++++++++++++++++++++++-------------
src/backend/utils/activity/pgstat.c | 2 +-
2 files changed, 32 insertions(+), 18 deletions(-)
