Re: New function pg_stat_statements_reset_query() to reset statistics of a specific query

From: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
To: Haribabu Kommi <kommi(dot)haribabu(at)gmail(dot)com>
Cc: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>, Michael Paquier <michael(at)paquier(dot)xyz>, Magnus Hagander <magnus(at)hagander(dot)net>, sk(at)zsrv(dot)org, Robert Haas <robertmhaas(at)gmail(dot)com>, Fujii Masao <masao(dot)fujii(at)gmail(dot)com>, Ashutosh Sharma <ashu(dot)coek88(at)gmail(dot)com>, Euler Taveira <euler(at)timbira(dot)com(dot)br>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Simon Riggs <simon(at)2ndquadrant(dot)com>, Dave Page <dpage(at)pgadmin(dot)org>
Subject: Re: New function pg_stat_statements_reset_query() to reset statistics of a specific query
Date: 2018-11-20 06:14:35
Message-ID: CAA4eK1LH-_rzLQ_PXmkNvGXum1V6BU7ziG5iqKjjaO2k3d5gDg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Tue, Nov 20, 2018 at 10:56 AM Haribabu Kommi
<kommi(dot)haribabu(at)gmail(dot)com> wrote:
>
> On Tue, Nov 20, 2018 at 2:06 PM Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>>
>> On Mon, Nov 19, 2018 at 10:48 AM Haribabu Kommi
>> <kommi(dot)haribabu(at)gmail(dot)com> wrote:
>> > On Mon, Nov 19, 2018 at 1:37 PM Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> wrote:
>> >>
>> >> On 2018-Nov-19, Michael Paquier wrote:
>> >>
>> >> > On Mon, Nov 19, 2018 at 10:41:22AM +1100, Haribabu Kommi wrote:
>> >> > > So 6 new functions needs to be added to cover all the above cases,
>> >> > > IMO, we may need functions for all combinations, because I feel some
>> >> > > user may have the requirement of left out combination, in case if we choose
>> >> > > only some combinations.
>> >> >
>> >> > That's bloating the interface in my opinion.
>> >>
>> >> I understand.
>> >>
>> >> Let's call for a vote from a larger audience. It's important to get
>> >> this interface right, ISTM.
>> >
>> > 4. Single API with -1 as invalid value, treat NULL as no matching. (Only problem
>> > with this approach is till now -1 is also a valid queryid, but setting -1 as queryid
>> > needs to be avoided.
>> >
>>
>> Hmm, can we use 0 as default value without any such caveat?
>
>
> Yes, with strict and 0 as default value can work.
> If there is no problem, I can go ahead with the above changes?
>

I would say wait for a few days (at least 2 days) to see if Alvaro or
anybody else has an opinion on this.

--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Takahashi, Ryohei 2018-11-20 06:20:45 Too many logs are written on Windows (LOG: could not reserve shared memory region (addr=%p) for child %p:)
Previous Message Haribabu Kommi 2018-11-20 05:26:33 Re: New function pg_stat_statements_reset_query() to reset statistics of a specific query