bookkeeper icon indicating copy to clipboard operation
bookkeeper copied to clipboard

ZkLedgerUnderreplicationManager: cache responses / reduce number of zk calls by isLedgerReplicationEnabled()

Open dlg99 opened this issue 3 years ago • 1 comments

It looks like ZkLedgerUnderreplicationManager.isLedgerReplicationEnabled() can cache the state and either use watcher (see notifyLedgerReplicationEnabled) + enableLedgerReplication/disableLedgerReplication or just some time interval (10sec or so) to cache the value for.

AFAICT Auditor.checkAllLedgers() calls it rather frequently

dlg99 avatar Jun 24 '22 18:06 dlg99

good idea.

horizonzy avatar Jun 26 '22 09:06 horizonzy