remove checking for client node on rivers, its automatically done with master and non river node anyhow

modules/elasticsearch/src/main/java/org/elasticsearch/river/cluster/PublishRiverClusterStateAction.java

@@ -72,10 +72,6 @@ public class PublishRiverClusterStateAction extends AbstractComponent {
             // master nodes because they will handle the state and the allocation of rivers
             // and river nodes since they will end up creating indexes
 
-            if (node.clientNode()) {
-                continue;
-            }
-
             if (!node.masterNode() && !RiverNodeHelper.isRiverNode(node)) {
                 continue;
             }

modules/elasticsearch/src/main/java/org/elasticsearch/river/cluster/RiverNodeHelper.java

@@ -28,6 +28,7 @@ import org.elasticsearch.river.RiverName;
 public class RiverNodeHelper {
 
     public static boolean isRiverNode(DiscoveryNode node) {
+        // we don't allocate rivers on client nodes
         if (node.clientNode()) {
             return false;
         }