You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@bookkeeper.apache.org by GitBox <gi...@apache.org> on 2022/06/24 18:28:46 UTC

[GitHub] [bookkeeper] dlg99 opened a new issue, #3358: ZkLedgerUnderreplicationManager: cache responses / reduce number of zk calls by isLedgerReplicationEnabled()

dlg99 opened a new issue, #3358:
URL: https://github.com/apache/bookkeeper/issues/3358

   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


-- 
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: issues-unsubscribe@bookkeeper.apache.org.apache.org

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

[GitHub] [bookkeeper] horizonzy commented on issue #3358: ZkLedgerUnderreplicationManager: cache responses / reduce number of zk calls by isLedgerReplicationEnabled()

Posted by GitBox <gi...@apache.org>.

horizonzy commented on issue #3358:
URL: https://github.com/apache/bookkeeper/issues/3358#issuecomment-1166465151

   good idea.


-- 
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: issues-unsubscribe@bookkeeper.apache.org

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