[GitHub] [accumulo] EdColeman commented on a diff in pull request #2778: Add version check to ZooPropStore

[GitBox <gi...@apache.org>]

EdColeman commented on code in PR #2778:
URL: https://github.com/apache/accumulo/pull/2778#discussion_r934578710


##########
server/base/src/main/java/org/apache/accumulo/server/conf/ServerConfigurationFactory.java:
##########
@@ -51,9 +65,18 @@ public class ServerConfigurationFactory extends ServerConfiguration {
   private final SiteConfiguration siteConfig;
   private final DeleteWatcher deleteWatcher = new DeleteWatcher();
 
+  private static final int REFRESH_PERIOD_MINUTES = 15;

Review Comment:
   The watcher should fire and update when the change is written to ZooKeeper - this is just to ensure that if a notification was missed this will make it (eventually) consistent.  Ideally, this would never find anything that is not up to date.



-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org