From: | Mahendra Singh Thalor <mahi6run(at)gmail(dot)com> |
---|---|
To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
Cc: | Andres Freund <andres(at)anarazel(dot)de>, Amit Kapila <akapila(at)postgresql(dot)org>, pgsql-committers(at)lists(dot)postgresql(dot)org |
Subject: | Re: pgsql: Allow vacuum command to process indexes in parallel. |
Date: | 2020-04-01 03:09:10 |
Message-ID: | CAKYtNAqTx2EgXQgsJjqVEHBaQr_9AEFxM82mTjXp4T=scVDFFw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-committers |
On Tue, 31 Mar 2020 at 17:28, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>
> On Mon, Mar 30, 2020 at 8:30 AM Mahendra Singh Thalor
> <mahi6run(at)gmail(dot)com> wrote:
> >
> > > > > 2077 if ((shared_balance >= VacuumCostLimit) &&
> > > > > >>> CID ...: Incorrect expression (UNINTENDED_INTEGER_DIVISION)
> > > > > >>> Dividing integer expressions "VacuumCostLimit" and "nworkers", and then converting the integer quotient to type "double". Any remainder, or fractional part of the quotient, is ignored.
> > > > > 2078 (VacuumCostBalanceLocal > 0.5 * (VacuumCostLimit / nworkers)))
> > > > > 2079 {
> > > > > 2080 /* Compute sleep time based on the local cost balance */
> > > > > 2081 msec = VacuumCostDelay * VacuumCostBalanceLocal / VacuumCostLimit;
> > > > > 2082 pg_atomic_sub_fetch_u32(VacuumSharedCostBalance, VacuumCostBalanceLocal);
> > > > > 2083 VacuumCostBalanceLocal = 0;
> > > >
> > > > Which seems like a fair enough complaint?
> > > >
> > >
> > > I'll look into it.
> > >
> >
> > Hi,
> > Attaching patch to fix this but I don't have coverity setup so I
> > haven't verified fix.
> >
>
> - (VacuumCostBalanceLocal > 0.5 * (VacuumCostLimit / nworkers)))
> + (VacuumCostBalanceLocal > (int) (0.5 * ((double) VacuumCostLimit /
> nworkers))))
> {
>
> I think typecasting to double should be enough to fix this coverity error.
I also think same so attaching updated patch.
--
Thanks and Regards
Mahendra Singh Thalor
EnterpriseDB: http://www.enterprisedb.com
Attachment | Content-Type | Size |
---|---|---|
v002-Fix-warning-in-compute_parallel_delay-for-Coverity.patch | application/octet-stream | 932 bytes |
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2020-04-01 04:07:25 | pgsql: Fix coverity complaint about commit 40d964ec99. |
Previous Message | Bruce Momjian | 2020-04-01 03:01:40 | pgsql: psql: do file completion for \gx |