You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by ap...@apache.org on 2013/12/18 02:43:29 UTC
svn commit: r1551800 -
/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
Author: apurtell
Date: Wed Dec 18 01:43:29 2013
New Revision: 1551800
URL: http://svn.apache.org/r1551800
Log:
HBASE-10179. HRegionServer underreports readRequestCounts by 1 under certain conditions (Perry Trolard)
Modified:
hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java?rev=1551800&r1=1551799&r2=1551800&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java Wed Dec 18 01:43:29 2013
@@ -2670,7 +2670,7 @@ public class HRegionServer implements HR
int i = 0;
synchronized(s) {
for (; i < nbRows
- && currentScanResultSize < maxScannerResultSize; i++) {
+ && currentScanResultSize < maxScannerResultSize; ) {
// Collect values to be returned here
boolean moreRows = s.nextRaw(values, SchemaMetrics.METRIC_NEXTSIZE);
if (!values.isEmpty()) {
@@ -2680,6 +2680,7 @@ public class HRegionServer implements HR
}
}
results.add(new Result(values));
+ i++;
}
if (!moreRows) {
break;