diff options
| author | Michael Paquier <michael@paquier.xyz> | 2019-09-04 15:46:37 +0900 |
|---|---|---|
| committer | Michael Paquier <michael@paquier.xyz> | 2019-09-04 15:46:37 +0900 |
| commit | ae060a52b2881ea842f596fa78b8d09f9a91b149 (patch) | |
| tree | bc2ee716c149dd24752daa693cfa4c6bb0339585 /src | |
| parent | 522baf14847a7e4cc97c49c7b1c28d21bc33921f (diff) | |
| download | postgresql-ae060a52b2881ea842f596fa78b8d09f9a91b149.tar.gz postgresql-ae060a52b2881ea842f596fa78b8d09f9a91b149.zip | |
Fix thinko when ending progress report for a backend
The logic ending progress reporting for a backend entry introduced by
b6fb647 causes callers of pgstat_progress_end_command() to do some extra
work when track_activities is enabled as the process fields are reset in
the backend entry even if no command were started for reporting.
This resets the fields only if a command is registered for progress
reporting, and only if track_activities is enabled.
Author: Masahiho Sawada
Discussion: https://postgr.es/m/CAD21AoCry_vJ0E-m5oxJXGL3pnos-xYGCzF95rK5Bbi3Uf-rpA@mail.gmail.com
Backpatch-through: 9.6
Diffstat (limited to 'src')
| -rw-r--r-- | src/backend/postmaster/pgstat.c | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/src/backend/postmaster/pgstat.c b/src/backend/postmaster/pgstat.c index d362e7f7d7d..011076c3e3e 100644 --- a/src/backend/postmaster/pgstat.c +++ b/src/backend/postmaster/pgstat.c @@ -3272,10 +3272,10 @@ pgstat_progress_end_command(void) { volatile PgBackendStatus *beentry = MyBEEntry; - if (!beentry) + if (!beentry || !pgstat_track_activities) return; - if (!pgstat_track_activities - && beentry->st_progress_command == PROGRESS_COMMAND_INVALID) + + if (beentry->st_progress_command == PROGRESS_COMMAND_INVALID) return; PGSTAT_BEGIN_WRITE_ACTIVITY(beentry); |