diff options
author | Jon Bratseth <bratseth@gmail.com> | 2022-03-09 12:53:02 +0100 |
---|---|---|
committer | Jon Bratseth <bratseth@gmail.com> | 2022-03-09 12:53:02 +0100 |
commit | f035afd9460684334860eac91698bc45a6558fbd (patch) | |
tree | 670b415a72ee8c22b1ee727a8f99112030831bcd /container-search | |
parent | b303b3a8c674c9f3f514a1747dc7abbf99907627 (diff) |
Cleanup
Diffstat (limited to 'container-search')
-rw-r--r-- | container-search/src/main/java/com/yahoo/search/dispatch/LoadBalancer.java | 8 |
1 files changed, 3 insertions, 5 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/dispatch/LoadBalancer.java b/container-search/src/main/java/com/yahoo/search/dispatch/LoadBalancer.java index 7934a9e25f6..4c0bcb38d15 100644 --- a/container-search/src/main/java/com/yahoo/search/dispatch/LoadBalancer.java +++ b/container-search/src/main/java/com/yahoo/search/dispatch/LoadBalancer.java @@ -85,6 +85,7 @@ public class LoadBalancer { } static class GroupStatus { + private final Group group; private int allocations = 0; private long queries = 0; @@ -233,11 +234,8 @@ public class LoadBalancer { public Optional<GroupStatus> takeNextGroup(Set<Integer> rejectedGroups) { double needle = random.nextDouble(); Optional<GroupStatus> gs = selectGroup(needle, true, rejectedGroups); - if (gs.isPresent()) { - return gs; - } - // fallback - any coverage better than none - return selectGroup(needle, false, rejectedGroups); + if (gs.isPresent()) return gs; + return selectGroup(needle, false, rejectedGroups); // any coverage better than none } } |