false is false, and true is true, can not be both

1 files changed, 7 insertions, 4 deletions

diff --git a/container-search/src/main/java/com/yahoo/search/dispatch/searchcluster/SearchCluster.java b/container-search/src/main/java/com/yahoo/search/dispatch/searchcluster/SearchCluster.java
index c17212b2481..a55a970e8ff 100644
--- a/container-search/src/main/java/com/yahoo/search/dispatch/searchcluster/SearchCluster.java
+++ b/container-search/src/main/java/com/yahoo/search/dispatch/searchcluster/SearchCluster.java
@@ -202,18 +202,21 @@ public class SearchCluster implements NodeManager<Node> {
         return localCorpusDispatchTarget;
     }
 
+    private void updateWorkingState(Node node, boolean isWorking) {
+        node.setWorking(isWorking);
+        updateVipStatusOnNodeChange(node, isWorking);
+    }
+
     /** Called by the cluster monitor when node state changes to working */
     @Override
     public void working(Node node) {
-        node.setWorking(true);
-        updateVipStatusOnNodeChange(node, true);
+        updateWorkingState(node, true);
     }
 
     /** Called by the cluster monitor when node state changes to failed */
     @Override
     public void failed(Node node) {
-        node.setWorking(false);
-        updateVipStatusOnNodeChange(node, true);
+        updateWorkingState(node, false);
     }
 
     private void updateSufficientCoverage(Group group, boolean sufficientCoverage) {