ndimiduk commented on a change in pull request #775: HBASE-23230 Enforce member
visibility in HRegionServer
URL: https://github.com/apache/hbase/pull/775#discussion_r341225818
##########
File path:
hbase-server/src/main/java/org/apache/hadoop/hbase/regionserver/HRegionServer.java
##########
@@ -1797,18 +1762,19 @@ protected void chore() {
}
}
- static class PeriodicMemStoreFlusher extends ScheduledChore {
- final HRegionServer server;
- final static int RANGE_OF_DELAY = 5 * 60; // 5 min in seconds
- final static int MIN_DELAY_TIME = 0; // millisec
+ private static class PeriodicMemStoreFlusher extends ScheduledChore {
Review comment:
Yeah perhaps in a subsequent patch. Trying to keep this scope narrow on "OOP
101" stuff.
----------------------------------------------------------------
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]
With regards,
Apache Git Services
