RE: Re: [Proposal] Add accumulated statistics for wait event

From: MyungKyu LIM <myungkyu(dot)lim(at)samsung(dot)com>
To: Alexander Korotkov <a(dot)korotkov(at)postgrespro(dot)ru>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Cc: Woosung Sohn <woosung(dot)sohn(at)samsung(dot)com>, DoHyung HONG <don(dot)hong(at)samsung(dot)com>
Subject: RE: Re: [Proposal] Add accumulated statistics for wait event
Date: 2018-07-24 10:10:33
Message-ID: 20180724101033epcms4p3da67d7bf66f4a75b0db65fc371ca2793@epcms4p3
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On Mon, Jul 23, 2018 at 10:53 AM Michael Paquier <michael(at)paquier(dot)xyz> wrote:
>> What's the performance penalty? I am pretty sure that this is
>> measurable as wait events are stored for a backend for each I/O
>> operation as well, and you are calling a C routine within an inlined
>> function which is designed to be light-weight, doing only a four-byte
>> atomic operation.

> Yes, the question is overhead of measuring durations of individual wait events. It has been proposed before, and there been heated debates about that (see threads [1-3]). It doesn't seem
> to be a conclusion about this feature. The thing to be said for sure:
> performance penalty heavily depends on OS/hardware/workload. In some cases overhead is negligible, but in other cases it appears to be huge.

Thanks for good information.
I agree. Performance penalty is exist.
But wait stats are demandable and useful. In some cases, it is worth sacrificing performance and using it.

So, what do you think about developing as extension? I have another concept proposal.
2. This feature can be implemented as extension if some hooks were provided in following functions,
- pgstat_report_wait_start
- pgstat_report_wait_end
This feature can be turned on/off by on-line config when necessary.

Best regards,
MyungKyu, Lim
 

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Eisentraut 2018-07-24 10:15:18 Re: psql \df option for procedures
Previous Message MyungKyu LIM 2018-07-24 10:06:23 RE: Re: [Proposal] Add accumulated statistics for wait event