You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2009/07/17 20:25:22 UTC

svn commit: r795183 - in /hadoop/hbase/branches/0.19: CHANGES.txt src/java/org/apache/hadoop/hbase/client/HConnectionManager.java src/java/org/apache/hadoop/hbase/client/ServerConnectionManager.java

Author: stack
Date: Fri Jul 17 18:25:21 2009
New Revision: 795183

URL: http://svn.apache.org/viewvc?rev=795183&view=rev
Log:
HBASE-1651 client is broken, it requests ROOT region location from ZK too much

Modified:
    hadoop/hbase/branches/0.19/CHANGES.txt
    hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/client/HConnectionManager.java
    hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/client/ServerConnectionManager.java

Modified: hadoop/hbase/branches/0.19/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.19/CHANGES.txt?rev=795183&r1=795182&r2=795183&view=diff
==============================================================================
--- hadoop/hbase/branches/0.19/CHANGES.txt (original)
+++ hadoop/hbase/branches/0.19/CHANGES.txt Fri Jul 17 18:25:21 2009
@@ -26,6 +26,8 @@
    HBASE-1583  Start/Stop of large cluster untenable
    HBASE-1609  We wait on leases to expire before regionserver goes down.
                Rather, just let client fail
+   HBASE-1651  client is broken, it requests ROOT region location from ZK
+               too much
 
 Release 0.19.3 - May 27th, 2009
   BUG FIXES

Modified: hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/client/HConnectionManager.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/client/HConnectionManager.java?rev=795183&r1=795182&r2=795183&view=diff
==============================================================================
--- hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/client/HConnectionManager.java (original)
+++ hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/client/HConnectionManager.java Fri Jul 17 18:25:21 2009
@@ -462,7 +462,7 @@
           // second waits. The second thread will not do find.
           
           if (!useCache || rootRegionLocation == null) {
-            return locateRootRegion();
+            rootRegionLocation = locateRootRegion();
           }
           return rootRegionLocation;
         }        

Modified: hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/client/ServerConnectionManager.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/client/ServerConnectionManager.java?rev=795183&r1=795182&r2=795183&view=diff
==============================================================================
--- hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/client/ServerConnectionManager.java (original)
+++ hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/client/ServerConnectionManager.java Fri Jul 17 18:25:21 2009
@@ -24,7 +24,7 @@
 
 /**
  * Used by server processes to expose HServerConnection method
- * setRootRegionLocation
+ * do can call {@link HConnectionManager#setRootRegionLocation}
  */
 public class ServerConnectionManager extends HConnectionManager {
   /*
@@ -41,4 +41,4 @@
   public static ServerConnection getConnection(HBaseConfiguration conf) {
     return (ServerConnection) HConnectionManager.getConnection(conf);
   }
-}
+}
\ No newline at end of file