You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by jd...@apache.org on 2011/10/17 21:01:58 UTC

svn commit: r1185327 - in /hbase/branches/0.92: CHANGES.txt src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java

Author: jdcryans
Date: Mon Oct 17 19:01:58 2011
New Revision: 1185327

URL: http://svn.apache.org/viewvc?rev=1185327&view=rev
Log:
HBASE-4606  Remove spam in HCM and fix a list.size == 0

Modified:
    hbase/branches/0.92/CHANGES.txt
    hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java

Modified: hbase/branches/0.92/CHANGES.txt
URL: http://svn.apache.org/viewvc/hbase/branches/0.92/CHANGES.txt?rev=1185327&r1=1185326&r2=1185327&view=diff
==============================================================================
--- hbase/branches/0.92/CHANGES.txt (original)
+++ hbase/branches/0.92/CHANGES.txt Mon Oct 17 19:01:58 2011
@@ -594,6 +594,7 @@ Release 0.92.0 - Unreleased
    HBASE-4558  Refactor TestOpenedRegionHandler and TestOpenRegionHandler. (Ram)
    HBASE-4558  Addendum for TestMasterFailOver (Ram) - Breaks the build
    HBASE-4568  Make zk dump jsp response faster
+   HBASE-4606  Remove spam in HCM and fix a list.size == 0
 
   TASKS
    HBASE-3559  Move report of split to master OFF the heartbeat channel

Modified: hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java?rev=1185327&r1=1185326&r2=1185327&view=diff
==============================================================================
--- hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java (original)
+++ hbase/branches/0.92/src/main/java/org/apache/hadoop/hbase/client/HConnectionManager.java Mon Oct 17 19:01:58 2011
@@ -1400,12 +1400,9 @@ public class HConnectionManager {
         throw new IllegalArgumentException(
             "argument results must be the same size as argument list");
       }
-      if (list.size() == 0) {
+      if (list.isEmpty()) {
         return;
       }
-      if (LOG.isDebugEnabled()) {
-        LOG.debug("expecting "+results.length+" results");
-      }
 
       // Keep track of the most recent servers for any given item for better
       // exceptional reporting.  We keep HRegionLocation to save on parsing.