hangc0276 commented on code in PR #4613:
URL: https://github.com/apache/bookkeeper/pull/4613#discussion_r2314831687
##########
bookkeeper-server/src/main/java/org/apache/bookkeeper/client/LedgerOpenOp.java:
##########
@@ -174,9 +195,21 @@ private void openWithMetadata(Versioned<LedgerMetadata>
versionedMetadata) {
}
// get the ledger metadata back
+ final boolean watchImmediately = !doRecovery || metadata.isClosed();
Review Comment:
We changed the behavior here. When the replication worker opens ledgers
without doRecovery, it may check all the ledgers in the BookKeeper cluster.
What's more, most of the ledgers are in a closed state. It will register a lot
of listeners and bring heavy pressure on Zookeeper.
--
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]
