ramkrish86 commented on a change in pull request #2483:
URL: https://github.com/apache/hbase/pull/2483#discussion_r521531128



##########
File path: 
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java
##########
@@ -3481,6 +3512,9 @@ public ScanResponse scan(final RpcController controller, 
final ScanRequest reque
       }
       throw new ServiceException(e);
     }
+    if(rsh.isFullRegionScan) {
+      rpcFullScanRequestCount.increment();

Review comment:
       Ya this is in RsRpcServices under scan() so should be ok. One thing I 
noticed is that - once  a scan is created and a next() call happens does the 
rpcScanRequestCount becomes 2 or is it 1?




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to