anoopsjohn commented on a change in pull request #2483:
URL: https://github.com/apache/hbase/pull/2483#discussion_r501662758
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RegionScanner.java
##########
@@ -115,4 +116,6 @@
*/
boolean nextRaw(List<Cell> result, ScannerContext scannerContext)
throws IOException;
+
+ boolean isFullRegionScan();
Review comment:
Agree. After seeing this wanted to say abt def impl in interface (at
least).. But ya these methods in CP exposed interfaces make more work for
phoenix kind of CPs which wraps the RegionScanner. Exposing the user scan or
its start/stop rows directly also not correct. We cleaned all these stuff in
2+ versions.
Why cant we do the new metric accounting in lower layer of HRegion ? not in
RS
----------------------------------------------------------------
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]
