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 2009/03/05 15:09:29 UTC

svn commit: r750463 - in /hadoop/hbase/trunk: CHANGES.txt src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

Author: jdcryans
Date: Thu Mar  5 14:09:29 2009
New Revision: 750463

URL: http://svn.apache.org/viewvc?rev=750463&view=rev
Log:
HBASE-1185  wrong request/sec in the gui reporting wrong

Modified:
    hadoop/hbase/trunk/CHANGES.txt
    hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

Modified: hadoop/hbase/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/CHANGES.txt?rev=750463&r1=750462&r2=750463&view=diff
==============================================================================
--- hadoop/hbase/trunk/CHANGES.txt (original)
+++ hadoop/hbase/trunk/CHANGES.txt Thu Mar  5 14:09:29 2009
@@ -31,6 +31,7 @@
    HBASE-1211  NPE in retries exhausted exception
    HBASE-1233  Transactional fixes: Overly conservative scan read-set,
                potential CME (Clint Morgan via Stack)
+   HBASE-1185  wrong request/sec in the gui reporting wrong
 
   IMPROVEMENTS
    HBASE-1089  Add count of regions on filesystem to master UI; add percentage

Modified: hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java?rev=750463&r1=750462&r2=750463&view=diff
==============================================================================
--- hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java (original)
+++ hadoop/hbase/trunk/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java Thu Mar  5 14:09:29 2009
@@ -1580,7 +1580,6 @@
 
   public RowResult[] next(final long scannerId, int nbRows) throws IOException {
     checkOpen();
-    requestCount.incrementAndGet();
     ArrayList<RowResult> resultSets = new ArrayList<RowResult>();
     try {
       String scannerName = String.valueOf(scannerId);
@@ -1590,6 +1589,7 @@
       }
       this.leases.renewLease(scannerName);
       for(int i = 0; i < nbRows; i++) {
+        requestCount.incrementAndGet();
         // Collect values to be returned here
         HbaseMapWritable<byte [], Cell> values
           = new HbaseMapWritable<byte [], Cell>();