Use actual backend IDs in pg_stat_get_backend_idset() and friends.
authorTom Lane <tgl@sss.pgh.pa.us>
Thu, 29 Sep 2022 16:14:39 +0000 (12:14 -0400)
committerTom Lane <tgl@sss.pgh.pa.us>
Thu, 29 Sep 2022 16:14:39 +0000 (12:14 -0400)
commitd7e39d72ca1c6f188b400d7d58813ff5b5b79064
treefad7ad7d202d0605df7f7bbc66a9dbf85d767204
parentd5e3fe682ac327ae2c0682a931f95e946ae13487
Use actual backend IDs in pg_stat_get_backend_idset() and friends.

Up to now, the ID values returned by pg_stat_get_backend_idset() and
used by pg_stat_get_backend_activity() and allied functions were just
indexes into a local array of sessions seen by the last stats refresh.
This is problematic for a few reasons.  The "ID" of a session can vary
over its existence, which is surprising.  Also, while these numbers
often match the "backend ID" used for purposes like temp schema
assignment, that isn't reliably true.  We can fairly cheaply switch
things around to make these numbers actually be the sessions' backend
IDs.  The added test case illustrates that with this definition, the
temp schema used by a given session can be obtained given its PID.

While here, delete some dead code that guarded against getting
a NULL return from pgstat_fetch_stat_local_beentry().  That can't
happen as long as the caller is careful to pass an in-range array
index, as all the callers are.  (This code may not have been dead
when written, but it surely is now.)

Nathan Bossart

Discussion: https://postgr.es/m/20220815205811.GA250990@nathanxps13
doc/src/sgml/monitoring.sgml
src/backend/utils/activity/backend_status.c
src/backend/utils/adt/pgstatfuncs.c
src/include/utils/backend_status.h
src/test/regress/expected/stats.out
src/test/regress/sql/stats.sql