summaryrefslogtreecommitdiffstats
path: root/container-search
diff options
context:
space:
mode:
authorHenning Baldersheim <balder@yahoo-inc.com>2022-08-22 09:24:52 +0200
committerHenning Baldersheim <balder@yahoo-inc.com>2022-08-22 09:24:52 +0200
commitb077873e79370f2e769c3ded133c52fb74a4ad48 (patch)
treef284cf3f663f273b631bf66218960df227321e32 /container-search
parent138c358b94e1a68fc10d1fa54b86952e57b7f5cb (diff)
Use an enhanced switch to ensure all branches are covered.
Diffstat (limited to 'container-search')
-rw-r--r--container-search/src/main/java/com/yahoo/search/dispatch/Dispatcher.java17
1 files changed, 6 insertions, 11 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/dispatch/Dispatcher.java b/container-search/src/main/java/com/yahoo/search/dispatch/Dispatcher.java
index 8bec018f8f8..a9a000e36e3 100644
--- a/container-search/src/main/java/com/yahoo/search/dispatch/Dispatcher.java
+++ b/container-search/src/main/java/com/yahoo/search/dispatch/Dispatcher.java
@@ -97,17 +97,12 @@ public class Dispatcher extends AbstractComponent {
}
private static LoadBalancer.Policy toLoadBalancerPolicy(DispatchConfig.DistributionPolicy.Enum policy) {
- if (policy == DispatchConfig.DistributionPolicy.ROUNDROBIN) {
- return LoadBalancer.Policy.ROUNDROBIN;
- } else if (policy == DispatchConfig.DistributionPolicy.BEST_OF_RANDOM_2) {
- return LoadBalancer.Policy.BEST_OF_RANDOM_2;
- } else if (policy == DispatchConfig.DistributionPolicy.LATENCY_AMORTIZED_OVER_REQUESTS) {
- return LoadBalancer.Policy.LATENCY_AMORTIZED_OVER_REQUESTS;
- } else if (policy == DispatchConfig.DistributionPolicy.LATENCY_AMORTIZED_OVER_TIME) {
- return LoadBalancer.Policy.LATENCY_AMORTIZED_OVER_TIME;
- } else {
- return LoadBalancer.Policy.LATENCY_AMORTIZED_OVER_REQUESTS;
- }
+ return switch (policy) {
+ case ROUNDROBIN: yield LoadBalancer.Policy.ROUNDROBIN;
+ case BEST_OF_RANDOM_2: yield LoadBalancer.Policy.BEST_OF_RANDOM_2;
+ case ADAPTIVE,LATENCY_AMORTIZED_OVER_REQUESTS: yield LoadBalancer.Policy.LATENCY_AMORTIZED_OVER_REQUESTS;
+ case LATENCY_AMORTIZED_OVER_TIME: yield LoadBalancer.Policy.LATENCY_AMORTIZED_OVER_TIME;
+ };
}
/* Protected for simple mocking in tests. Beware that searchCluster is shutdown on in deconstruct() */