diff options
| author | Jon Bratseth <bratseth@gmail.com> | 2023-10-26 13:20:31 +0300 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-10-26 13:20:31 +0300 |
| commit | 0f3fc1c12d97f92e7826e43af2ad1740a8390e17 (patch) | |
| tree | 0673dd59a2f83cd904d3e5ee3c23542f6e3ff58c | |
| parent | be43a57b172ec346d1615348977ad45959a4db1c (diff) | |
| parent | 2c195507e723d0fd932222f1311ca403ea5bd7f7 (diff) | |
Merge pull request #29112 from vespa-engine/bjormel/handle_total_qps_0
Handle groupQps = 0
| -rw-r--r-- | controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/BcpGroupUpdater.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/BcpGroupUpdater.java b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/BcpGroupUpdater.java index 36aeb598890..92aaacaa1f0 100644 --- a/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/BcpGroupUpdater.java +++ b/controller-server/src/main/java/com/yahoo/vespa/hosted/controller/maintenance/BcpGroupUpdater.java @@ -121,7 +121,9 @@ public class BcpGroupUpdater extends ControllerMaintainer { currentReadShare += groupQps == 0 ? 0 : fraction * deploymentQps / groupQps; maxReadShare += group.size() == 1 ? currentReadShare - : fraction * ( deploymentQps + group.maxQpsExcluding(deployment.zone().region()) / (group.size() - 1) ) / groupQps; + : groupQps != 0 + ? fraction * (deploymentQps + group.maxQpsExcluding(deployment.zone().region()) / (group.size() - 1)) / groupQps + : 0; } patch.currentReadShare = currentReadShare; patch.maxReadShare = maxReadShare; |