[GitHub] jvrao commented on a change in pull request #1902: Metadata checker validating EnsemblePlacementpolicy

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

jvrao commented on a change in pull request #1902: Metadata checker validating EnsemblePlacementpolicy
URL: https://github.com/apache/bookkeeper/pull/1902#discussion_r251217182
 
 

 ##########
 File path: bookkeeper-server/src/main/java/org/apache/bookkeeper/conf/ServerConfiguration.java
 ##########
 @@ -2188,6 +2189,29 @@ public long getAuditorPeriodicBookieCheckInterval() {
         return getLong(AUDITOR_PERIODIC_BOOKIE_CHECK_INTERVAL, 86400);
     }
 
+    /**
+     * Set the regularity at which the auditor will run a metadata check of all
+     * write closed/fenced ledgers. This should not be run very often, and at
+     * most, once a day. Setting this to 0 will completely disable the periodic
+     * metadata check.
 
 Review comment:
   I think it makes lot of sense to have a way to do ensemble placement check independently. This is light weight and more pone to errors.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services