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:08:19 UTC

svn commit: r750462 - in /hadoop/hbase/branches/0.19: CHANGES.txt src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java

Author: jdcryans
Date: Thu Mar  5 14:08:18 2009
New Revision: 750462

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

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

Modified: hadoop/hbase/branches/0.19/CHANGES.txt
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.19/CHANGES.txt?rev=750462&r1=750461&r2=750462&view=diff
==============================================================================
--- hadoop/hbase/branches/0.19/CHANGES.txt (original)
+++ hadoop/hbase/branches/0.19/CHANGES.txt Thu Mar  5 14:08:18 2009
@@ -17,6 +17,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-845   HCM.isTableEnabled doesn't really tell if it is, or not

Modified: hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
URL: http://svn.apache.org/viewvc/hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java?rev=750462&r1=750461&r2=750462&view=diff
==============================================================================
--- hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java (original)
+++ hadoop/hbase/branches/0.19/src/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java Thu Mar  5 14:08:18 2009
@@ -1566,7 +1566,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);
@@ -1576,6 +1575,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>();