pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Amit Kapila <akapila@postgresql.org>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Track last_inactive_time in pg_replication_slots.
Date: Mon, 25 Mar 2024 11:11:27 +0000
Message-ID: <E1roiEh-005Z7e-88@gemulon.postgresql.org> (raw)

Track last_inactive_time in pg_replication_slots.

This commit adds a new property called last_inactive_time for slots. It is
set to 0 whenever a slot is made active/acquired and set to the current
timestamp whenever the slot is inactive/released or restored from the disk.
Note that we don't set the last_inactive_time for the slots currently being
synced from the primary to the standby because such slots are typically
inactive as decoding is not allowed on those.

The 'last_inactive_time' will be useful on production servers to debug and
analyze inactive replication slots. It will also help to know the lifetime
of a replication slot - one can know how long a streaming standby, logical
subscriber, or replication slot consumer is down.

The 'last_inactive_time' will also be useful to implement inactive
timeout-based replication slot invalidation in a future commit.

Author: Bharath Rupireddy
Reviewed-by: Bertrand Drouvot, Amit Kapila, Shveta Malik
Discussion: https://www.postgresql.org/message-id/CALj2ACW4aUe-_uFQOjdWCEN-xXoLGhmvRFnL8SNw_TZ5nJe+aw@mail.gmail...

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/a11f330b5584f2430371d68871e00f5c63735299

Modified Files
--------------
doc/src/sgml/system-views.sgml            |  10 ++
src/backend/catalog/system_views.sql      |   1 +
src/backend/replication/slot.c            |  35 +++++++
src/backend/replication/slotfuncs.c       |   7 +-
src/include/catalog/catversion.h          |   2 +-
src/include/catalog/pg_proc.dat           |   6 +-
src/include/replication/slot.h            |   3 +
src/test/recovery/t/019_replslot_limit.pl | 152 ++++++++++++++++++++++++++++++
src/test/regress/expected/rules.out       |   3 +-
9 files changed, 213 insertions(+), 6 deletions(-)



view thread (34+ messages)  latest in thread

Message-ID: <E1roiEh-005Z7e-88@gemulon.postgresql.org>
Permalink:  ../E1roiEh-005Z7e-88@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1roiEh-005Z7e-88@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: akapila@postgresql.org, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Track last_inactive_time in pg_replication_slots.
  In-Reply-To: <E1roiEh-005Z7e-88@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox