Descriptions of the changes in this PR: - store last checkAllLedgers executed time in metadataserver. - use that value for determining the initialDelay of checkAllLedgers call - add dedicated metric for bookieCheck - add more log lines in Auditor
### Motivation In Auditor checkAllLedgers is scheduled to run for every ‘auditorPeriodicCheckInterval’ and with initial delay of ‘auditorPeriodicCheckInterval’. But the problem is if Auditor role keeps transforming to another AR process, it might not get chance to do ‘checkAllLedgers’ task. Considering default value for 'auditorPeriodicCheckInterval' is 1 week, it is not uncommon situation. ### Changes Thats why I'm planning to persist last successful checkAllLedgers execution ctime in ZK, so that next time when a new AR becomes Auditor, it would use that as reference to decide the initial delay value. [ Full content available at: https://github.com/apache/bookkeeper/pull/1683 ] This message was relayed via gitbox.apache.org for [email protected]
