Only use direct dispatch when the local node is responding

1 files changed, 7 insertions, 4 deletions

diff --git a/container-search/src/main/java/com/yahoo/search/cluster/NodeManager.java b/container-search/src/main/java/com/yahoo/search/cluster/NodeManager.java
index ce221fa1479..25582e43f5e 100644
--- a/container-search/src/main/java/com/yahoo/search/cluster/NodeManager.java
+++ b/container-search/src/main/java/com/yahoo/search/cluster/NodeManager.java
@@ -12,12 +12,15 @@ import java.util.concurrent.Executor;
 public interface NodeManager<T> {
 
     /** Called when a failed node is working (ready for production) again */
-    public void working(T node);
+    void working(T node);
 
     /** Called when a working node fails */
-    public void failed(T node);
+    void failed(T node);
 
-    /** Called when a node should be pinged */
-    public void ping(T node, Executor executor);
+    /** 
+     * Called when a node should be pinged. 
+     * This *must* lead to either a call to NodeMonitor.failed or NodeMonitor.responded 
+     */
+    void ping(T node, Executor executor);
 
 }