diff options
| author | Amit Kapila <akapila@postgresql.org> | 2020-04-01 09:28:13 +0530 |
|---|---|---|
| committer | Amit Kapila <akapila@postgresql.org> | 2020-04-01 09:28:13 +0530 |
| commit | 2401d93718310237b3cb1ff914abc1bcbdd8e1dc (patch) | |
| tree | 5c04ee18d5a4aa73606bbac477d200d2e26e5ae9 | |
| parent | 08481eedd186ea5c22eef86e85cacacbc715f995 (diff) | |
| download | postgresql-2401d93718310237b3cb1ff914abc1bcbdd8e1dc.tar.gz postgresql-2401d93718310237b3cb1ff914abc1bcbdd8e1dc.zip | |
Fix coverity complaint about commit 40d964ec99.
The coverity complained that dividing integer expressions and then
converting the integer quotient to type "double" would lose fractional
part. Typecasting one of the arguments of expression with double should
fix the report.
Author: Mahendra Singh Thalor
Reviewed-by: Amit Kapila
Discussion: https://postgr.es/m/20200329224818.6phnhv7o2q2rfovf@alap3.anarazel.de
| -rw-r--r-- | src/backend/commands/vacuum.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/backend/commands/vacuum.c b/src/backend/commands/vacuum.c index 59731d687f6..3a89f8fe1e2 100644 --- a/src/backend/commands/vacuum.c +++ b/src/backend/commands/vacuum.c @@ -2077,7 +2077,7 @@ compute_parallel_delay(void) VacuumCostBalanceLocal += VacuumCostBalance; if ((shared_balance >= VacuumCostLimit) && - (VacuumCostBalanceLocal > 0.5 * (VacuumCostLimit / nworkers))) + (VacuumCostBalanceLocal > 0.5 * ((double) VacuumCostLimit / nworkers))) { /* Compute sleep time based on the local cost balance */ msec = VacuumCostDelay * VacuumCostBalanceLocal / VacuumCostLimit; |