You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by te...@apache.org on 2013/01/15 04:21:55 UTC

svn commit: r1433274 - /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java

Author: tedyu
Date: Tue Jan 15 03:21:54 2013
New Revision: 1433274

URL: http://svn.apache.org/viewvc?rev=1433274&view=rev
Log:
HBASE-7562 ZKUtil: missing "else condition" in multi processing (Himanshu)


Modified:
    hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java

Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java?rev=1433274&r1=1433273&r2=1433274&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java Tue Jan 15 03:21:54 2013
@@ -1428,19 +1428,19 @@ public class ZKUtil {
       } catch (InterruptedException ie) {
         zkw.interruptedException(ie);
       }
-    }
-
-    // run sequentially
-    for (ZKUtilOp op : ops) {
-      if (op instanceof CreateAndFailSilent) {
-        createAndFailSilent(zkw, (CreateAndFailSilent)op);
-      } else if (op instanceof DeleteNodeFailSilent) {
-        deleteNodeFailSilent(zkw, (DeleteNodeFailSilent)op);
-      } else if (op instanceof SetData) {
-        setData(zkw, (SetData)op);
-      } else {
-        throw new UnsupportedOperationException("Unexpected ZKUtilOp type: "
-          + op.getClass().getName());
+    } else {
+      // run sequentially
+      for (ZKUtilOp op : ops) {
+        if (op instanceof CreateAndFailSilent) {
+          createAndFailSilent(zkw, (CreateAndFailSilent) op);
+        } else if (op instanceof DeleteNodeFailSilent) {
+          deleteNodeFailSilent(zkw, (DeleteNodeFailSilent) op);
+        } else if (op instanceof SetData) {
+          setData(zkw, (SetData) op);
+        } else {
+          throw new UnsupportedOperationException("Unexpected ZKUtilOp type: "
+              + op.getClass().getName());
+        }
       }
     }
   }