svn commit: r1081427 - in /hbase/trunk: CHANGES.txt src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java

[stack]

Author: stack
Date: Mon Mar 14 15:40:34 2011
New Revision: 1081427

URL: http://svn.apache.org/viewvc?rev=1081427&view=rev
Log:
HBASE-3633 ZKUtil::createSetData should only create a node when it nonexists

Modified:
    hbase/trunk/CHANGES.txt
    hbase/trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java

Modified: hbase/trunk/CHANGES.txt
URL: 
http://svn.apache.org/viewvc/hbase/trunk/CHANGES.txt?rev=1081427&r1=1081426&r2=1081427&view=diff
==============================================================================
--- hbase/trunk/CHANGES.txt (original)
+++ hbase/trunk/CHANGES.txt Mon Mar 14 15:40:34 2011
@@ -147,6 +147,8 @@ Release 0.90.2 - Unreleased
    HBASE-3582  Allow HMaster and HRegionServer to login from keytab
                hen on secure Hadoop
    HBASE-3608  MemstoreFlusher error message doesnt include exception!
+   HBASE-3633  ZKUtil::createSetData should only create a node when it
+               nonexists (Guanpeng Xu via Stack)
 
   IMPROVEMENTS
    HBASE-3542  MultiGet methods in Thrift

Modified: 
hbase/trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
URL: 
http://svn.apache.org/viewvc/hbase/trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java?rev=1081427&r1=1081426&r2=1081427&view=diff
==============================================================================
--- hbase/trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java 
(original)
+++ hbase/trunk/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java Mon 
Mar 14 15:40:34 2011
@@ -723,7 +723,7 @@ public class ZKUtil {
   public static void createSetData(final ZooKeeperWatcher zkw, final String 
znode,
       final byte [] data)
   throws KeeperException {
-    if (checkExists(zkw, znode) != -1) {
+    if (checkExists(zkw, znode) == -1) {
       ZKUtil.createWithParents(zkw, znode);
     }
     ZKUtil.setData(zkw, znode, data);