You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@helix.apache.org by ka...@apache.org on 2014/04/06 07:23:08 UTC

git commit: [HELIX-413] Removed empty resource check in ClusterStateVerifier

Repository: helix
Updated Branches:
  refs/heads/master 0ae690e01 -> fdc8e5063


[HELIX-413] Removed empty resource check in ClusterStateVerifier


Project: http://git-wip-us.apache.org/repos/asf/helix/repo
Commit: http://git-wip-us.apache.org/repos/asf/helix/commit/fdc8e506
Tree: http://git-wip-us.apache.org/repos/asf/helix/tree/fdc8e506
Diff: http://git-wip-us.apache.org/repos/asf/helix/diff/fdc8e506

Branch: refs/heads/master
Commit: fdc8e506368a47fa1836b7f3d888a8229749e186
Parents: 0ae690e
Author: Kanak Biscuitwala <ka...@hotmail.com>
Authored: Sat Apr 5 22:21:01 2014 -0700
Committer: Kanak Biscuitwala <ka...@hotmail.com>
Committed: Sat Apr 5 22:21:01 2014 -0700

----------------------------------------------------------------------
 .../src/main/java/org/apache/helix/tools/ClusterStateVerifier.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/helix/blob/fdc8e506/helix-core/src/main/java/org/apache/helix/tools/ClusterStateVerifier.java
----------------------------------------------------------------------
diff --git a/helix-core/src/main/java/org/apache/helix/tools/ClusterStateVerifier.java b/helix-core/src/main/java/org/apache/helix/tools/ClusterStateVerifier.java
index 3da9250..eec7ab5 100644
--- a/helix-core/src/main/java/org/apache/helix/tools/ClusterStateVerifier.java
+++ b/helix-core/src/main/java/org/apache/helix/tools/ClusterStateVerifier.java
@@ -257,7 +257,7 @@ public class ClusterStateVerifier {
       }
 
       // Filter resources if requested
-      if (resources != null && !resources.isEmpty()) {
+      if (resources != null) {
         idealStates.keySet().retainAll(resources);
         extViews.keySet().retainAll(resources);
       }