jvrao commented on a change in pull request #1902: (WIP) Metadata checker
validating EnsemblePlacementpolicy
URL: https://github.com/apache/bookkeeper/pull/1902#discussion_r248540776
##########
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
Review comment:
No need to say write closed -> as there is no read open / close ledger
state.
Also fenced ledger may not have final consistent state yet. Why do you need
to include
fenced ledgers, it is just a transient state right?
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
