external
/
pg_stat_monitor
mirror of https://github.com/percona/pg_stat_monitor.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Issue - (#53): Fix a bug where the query response time histogram does show proper counters. 

PG-143 : PostgreSQL-13 has a different issue so, special handling for PostgreSQL-13 done.
pull/55/head
Ibrar Ahmed 2020-10-12 22:53:37 +00:00
parent ec1ff594b4
commit bc47a0074b
1 changed files with 10 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
pg_stat_monitor.c
View File

@ -826,16 +826,20 @@ static void pgss_store(const char *query, uint64 queryId,
e->counters.time[kind].max_time = total_time; e->counters.time[kind].max_time = total_time;
} }
for (i = 0; i < MAX_RESPONSE_BUCKET - 1; i++) /* increment only in case of PGSS_EXEC */
if (kind == PGSS_EXEC)
{ {
if (total_time < PGSM_RESPOSE_TIME_LOWER_BOUND + (PGSM_RESPOSE_TIME_STEP * i)) for (i = 0; i < MAX_RESPONSE_BUCKET - 1; i++)
{ {
e->counters.resp_calls[i]++; if (total_time < PGSM_RESPOSE_TIME_LOWER_BOUND + (PGSM_RESPOSE_TIME_STEP * i))
break; {
e->counters.resp_calls[i]++;
break;
}
} }
} if (total_time > PGSM_RESPOSE_TIME_LOWER_BOUND + (PGSM_RESPOSE_TIME_STEP * MAX_RESPONSE_BUCKET))
if (total_time > PGSM_RESPOSE_TIME_LOWER_BOUND + (PGSM_RESPOSE_TIME_STEP * MAX_RESPONSE_BUCKET))
e->counters.resp_calls[MAX_RESPONSE_BUCKET - 1]++; e->counters.resp_calls[MAX_RESPONSE_BUCKET - 1]++;
}
e->counters.calls[kind].rows += rows; e->counters.calls[kind].rows += rows;
e->counters.blocks.shared_blks_hit += bufusage->shared_blks_hit; e->counters.blocks.shared_blks_hit += bufusage->shared_blks_hit;