hgromer commented on code in PR #6055:
URL: https://github.com/apache/hbase/pull/6055#discussion_r1742722583
##########
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/RSRpcServices.java:
##########
@@ -418,6 +418,18 @@ public void run() {
}
}
+ static class QuotaAndRegionScannerHolder {
+ private final String scannerName;
+ private final RegionScannerHolder rsh;
+ private final OperationQuota quota;
+
+ QuotaAndRegionScannerHolder(String scannerName, RegionScannerHolder rsh,
OperationQuota quota) {
Review Comment:
The only oddity here is that now the context may hold a Quota that isn't
up-to-date because updating the RegionScannerContext's quota isn't thread-safe.
There are other variables of the RSX that are updated this way though, so maybe
this isn't an actual issue.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]