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:33:37 UTC
svn commit: r1551795 -
/hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
Author: apurtell
Date: Wed Dec 18 01:33:36 2013
New Revision: 1551795
URL: http://svn.apache.org/r1551795
Log:
HBASE-10179. HRegionServer underreports readRequestCounts by 1 under certain conditions (Perry Trolard)
Modified:
hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
Modified: hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java?rev=1551795&r1=1551794&r2=1551795&view=diff
==============================================================================
--- hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java (original)
+++ hbase/branches/0.98/hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java Wed Dec 18 01:33:36 2013
@@ -3121,7 +3121,7 @@ public class HRegionServer implements Cl
int i = 0;
synchronized(scanner) {
for (; i < rows
- && currentScanResultSize < maxResultSize; i++) {
+ && currentScanResultSize < maxResultSize; ) {
// Collect values to be returned here
boolean moreRows = scanner.nextRaw(values);
if (!values.isEmpty()) {
@@ -3131,6 +3131,7 @@ public class HRegionServer implements Cl
}
}
results.add(Result.create(values));
+ i++;
}
if (!moreRows) {
break;