diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2022-08-22 09:24:52 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2022-08-22 09:24:52 +0200 |
commit | b077873e79370f2e769c3ded133c52fb74a4ad48 (patch) | |
tree | f284cf3f663f273b631bf66218960df227321e32 /container-search | |
parent | 138c358b94e1a68fc10d1fa54b86952e57b7f5cb (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.java | 17 |
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() */ |