bbeaudreault opened a new pull request, #4567: URL: https://github.com/apache/hbase/pull/4567
This is a relatively simple change -- We were already fetching ClusterMetrics. This change adds `Option.TABLE_TO_REGIONS_COUNT` to that and then uses `clusterMetrics.getTableRegionStatesCount()` instead of MetaTableAccessor. I couldn't think of any new tests to add here. There are substantial tests in TestClusterScopeQuotaThrottle, TestQuotaThrottle, and TestQuotaAdmin. I did some messing with the logic to prove that existing tests would have failed if there were a bug here. This is just a different implementation for an existing functionality, rather than a net new functionality to test. -- 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]
