From: | Andres Freund <andres(at)anarazel(dot)de> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: pgstat: prepare APIs used by pgstatfuncs for shared memory stats |
Date: | 2022-04-07 00:57:04 |
Message-ID: | E1ncGSN-000emw-L7@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
pgstat: prepare APIs used by pgstatfuncs for shared memory stats.
With the introduction of PgStat_Kind PgStat_Single_Reset_Type,
PgStat_Shared_Reset_Target don't make sense anymore. Replace them with
PgStat_Kind.
Instead of having dedicated reset functions for different kinds of stats, use
two generic helper routines (one to reset all stats of a kind, one to reset
one stats entry).
A number of reset functions were named pgstat_reset_*_counter(), despite
affecting multiple counters. The generic helper routines get rid of
pgstat_reset_single_counter(), pgstat_reset_subscription_counter().
Rename pgstat_reset_slru_counter(), pgstat_reset_replslot_counter() to
pgstat_reset_slru(), pgstat_reset_replslot() respectively, and have them only
deal with a single SLRU/slot. Resetting all SLRUs/slots goes through the
generic pgstat_reset_of_kind().
Previously pg_stat_reset_replication_slot() used SearchNamedReplicationSlot()
to check if a slot exists. API wise it seems better to move that to
pgstat_replslot.c.
This is done separately from the - quite large - shared memory statistics
patch to make review easier.
Reviewed-By: Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>
Discussion: https://postgr.es/m/20220404041516.cctrvpadhuriawlq@alap3.anarazel.de
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/8fb580a35ce358063dfdd10991d017498283c767
Modified Files
--------------
src/backend/postmaster/pgstat.c | 128 ++++++++++++++++-------
src/backend/utils/activity/pgstat_replslot.c | 37 +++++--
src/backend/utils/activity/pgstat_slru.c | 8 +-
src/backend/utils/activity/pgstat_subscription.c | 21 ----
src/backend/utils/adt/pgstatfuncs.c | 68 ++++++------
src/include/pgstat.h | 28 ++---
src/tools/pgindent/typedefs.list | 2 -
7 files changed, 161 insertions(+), 131 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2022-04-07 02:07:46 | pgsql: pgstat: revise replication slot API in preparation for shared me |
Previous Message | Michael Paquier | 2022-04-06 23:58:08 | Re: pgsql: pgstat: add pgstat_copy_relation_stats(). |