diff options
| author | Bruce Momjian <bruce@momjian.us> | 2021-04-08 11:16:01 -0400 |
|---|---|---|
| committer | Bruce Momjian <bruce@momjian.us> | 2021-04-08 11:16:01 -0400 |
| commit | 0f61727b75b93915ca9a9f20c996ed7020996a38 (patch) | |
| tree | edde84ea5ff138952c506bbffe93baa848c79fb8 /src/backend/utils/adt/pgstatfuncs.c | |
| parent | 5844c23dc50508aefeb8183be45f4ee99e9dec17 (diff) | |
| download | postgresql-0f61727b75b93915ca9a9f20c996ed7020996a38.tar.gz postgresql-0f61727b75b93915ca9a9f20c996ed7020996a38.zip | |
Fixes for query_id feature
Ignore parallel workers in pg_stat_statements
Oversight in 4f0b0966c8 which exposed queryid in parallel workers.
Counters are aggregated by the main backend process so parallel workers
would report duplicated activity, and could also report activity for the
wrong entry as they are only aware of the top level queryid.
Fix thinko in pg_stat_get_activity when retrieving the queryid.
Remove unnecessary call to pgstat_report_queryid().
Reported-by: Amit Kapila, Andres Freund, Thomas Munro
Discussion: https://postgr.es/m/20210408051735.lfbdzun5zdlax5gd@alap3.anarazel.de p634GTSOqnDW86Owrn6qDAVosC5dJjXjp7BMfc5Gz1Q@mail.gmail.com
Author: Julien Rouhaud
Diffstat (limited to 'src/backend/utils/adt/pgstatfuncs.c')
| -rw-r--r-- | src/backend/utils/adt/pgstatfuncs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/utils/adt/pgstatfuncs.c b/src/backend/utils/adt/pgstatfuncs.c index 9fa4a93162f..182b15e3f23 100644 --- a/src/backend/utils/adt/pgstatfuncs.c +++ b/src/backend/utils/adt/pgstatfuncs.c @@ -917,7 +917,7 @@ pg_stat_get_activity(PG_FUNCTION_ARGS) if (beentry->st_queryid == 0) nulls[29] = true; else - values[29] = DatumGetUInt64(beentry->st_queryid); + values[29] = UInt64GetDatum(beentry->st_queryid); } else { |