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/03/07 05:08:57 UTC

svn commit: r1453679 - /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureUtil.java

Author: tedyu
Date: Thu Mar  7 04:08:57 2013
New Revision: 1453679

URL: http://svn.apache.org/r1453679
Log:
HBASE-8019 Port HBASE-7779 '[snapshot 130201 merge] Fix TestMultiParallel' to 0.94 (Ted Yu)


Modified:
    hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureUtil.java

Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureUtil.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureUtil.java?rev=1453679&r1=1453678&r2=1453679&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureUtil.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/procedure/ZKProcedureUtil.java Thu Mar  7 04:08:57 2013
@@ -99,9 +99,8 @@ public abstract class ZKProcedureUtil
 
   @Override
   public void close() throws IOException {
-    if (watcher != null) {
-      watcher.close();
-    }
+    // the watcher is passed from either Master or Region Server
+    // watcher.close() will be called by the owner so no need to call close() here
   }
 
   public String getAcquiredBarrierNode(String opInstanceName) {
@@ -283,4 +282,4 @@ public abstract class ZKProcedureUtil
     ZKUtil.deleteNodeRecursively(watcher, getReachedBarrierNode(procedureName));
     ZKUtil.deleteNodeRecursively(watcher, getAbortZNode(procedureName));
   }
-}
\ No newline at end of file
+}