From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | pgsql: Make walsenders show their replication commands in pg_stat_activ |
Date: | 2021-06-25 14:46:33 |
Message-ID: | E1lwn6H-0003iZ-20@gemulon.postgresql.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
Make walsenders show their replication commands in pg_stat_activity.
A walsender process that has executed a SQL command left the text of
that command in pg_stat_activity.query indefinitely, which is quite
confusing if it's in RUNNING state but not doing that query. An easy
and useful fix is to treat replication commands as if they were SQL
queries, and show them in pg_stat_activity according to the same rules
as for regular queries. While we're at it, it seems also sensible to
set debug_query_string, allowing error logging and debugging to see
the replication command.
While here, clean up assorted silliness in exec_replication_command:
* Clean up SQLCmd code path, and fix its only-accidentally-not-buggy
memory management.
* Remove useless duplicate call of SnapBuildClearExportedSnapshot().
* replication_scanner_finish() was never called.
Back-patch of commit f560209c6 into v10-v13. I'd originally felt
that this didn't merit back-patching, but subsequent confusion
while debugging walsender problems suggests that it'll be useful.
Also, the original commit has now aged long enough to provide some
comfort that it won't cause problems.
Discussion: https://postgr.es/m/2673480.1624557299@sss.pgh.pa.us
Discussion: https://postgr.es/m/880181.1600026471@sss.pgh.pa.us
Branch
------
REL_12_STABLE
Details
-------
https://git.postgresql.org/pg/commitdiff/04a47605727753de40d5ecbc4f2e7dc38577a7c2
Modified Files
--------------
src/backend/replication/walsender.c | 66 ++++++++++++++++++++-----------------
1 file changed, 36 insertions(+), 30 deletions(-)
From | Date | Subject | |
---|---|---|---|
Next Message | Tom Lane | 2021-06-25 15:18:21 | pgsql: Doc: remove commit f560209c6 from v14 release notes. |
Previous Message | Michael Paquier | 2021-06-25 12:24:23 | Re: pgsql: Fix pattern matching logic for logs in TAP tests of pgbench |