wenbingshen commented on PR #3561:
URL: https://github.com/apache/bookkeeper/pull/3561#issuecomment-1286396491
> This brings another question: we also don't see what's going on in this
case (independent of this change). E.g. pausing AR for some time (maintenance
window etc.) is fine but is it ok to not update counts of underreplicated
ledgers?
@dlg99 Good catch. Thanks for your review and comments. :)
AutoRecovery's responsibilities may need to be divided into the following
departments:
Auditor: 1. Different ledger status detection.
2. and initiate ledger recovery tasks.
ReplicationWorker: Execute tasks triggered by Auditor.
maybe disable AutoRecovery needs to be divided into 2 parts:
1. Completely prohibit AutoRecovery work, including Auditor,
ReplicationWorker
2. Only the replication work of AutoRecovery is prohibited, and the Auditor
still performs the detection work and updates the relevant indicators.
This should be a new feature, we need other discussions and PRs.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
