Re: Data type correction in pgstat_report_replslot function parameters - Mailing list pgsql-hackers

From vignesh C
Subject Re: Data type correction in pgstat_report_replslot function parameters
Date
Msg-id CALDaNm27C8ODptcoHu_GB=qPaT=TpXHyuZJw6dMoQsUN00Vq+g@mail.gmail.com
Whole thread Raw
In response to Re: Data type correction in pgstat_report_replslot function parameters  (Fujii Masao <masao.fujii@oss.nttdata.com>)
Responses Re: Data type correction in pgstat_report_replslot function parameters  (Fujii Masao <masao.fujii@oss.nttdata.com>)
List pgsql-hackers
On Thu, Apr 1, 2021 at 11:18 PM Fujii Masao <masao.fujii@oss.nttdata.com> wrote:
>
>
>
> On 2021/04/02 2:18, Jeevan Ladhe wrote:
> >
> >
> > On Thu, Apr 1, 2021 at 10:20 PM vignesh C <vignesh21@gmail.com <mailto:vignesh21@gmail.com>> wrote:
> >
> >     Hi,
> >
> >     While I was reviewing replication slot statistics code, I found one
> >     issue in the data type used for pgstat_report_replslot function
> >     parameters. We pass int64 variables to the function but the function
> >     prototype uses int type. I I felt the function parameters should be
> >     int64. Attached patch fixes the same.
>
> Isn't it better to use PgStat_Counter instead of int64?
>

Thanks for your comment, the updated patch contains the changes for it.

Regards,
Vignesh

Attachment

pgsql-hackers by date:

Previous
From: David Rowley
Date:
Subject: Re: Hybrid Hash/Nested Loop joins and caching results from subplans
Next
From: "'alvherre@alvh.no-ip.org'"
Date:
Subject: Re: libpq debug log