shustsud commented on PR #4194: URL: https://github.com/apache/bookkeeper/pull/4194#issuecomment-1965865558
@graysonzeng > If this PR is merged, I think the repair of #4171 may be improved. Is the issue with https://github.com/apache/bookkeeper/pull/4171 that BookKeeperClientWorker-OrderedExecutor thread and pulsar-io thread called LedgerHandle#sendAddSuccessCallbacks at the same time? Even if this PR is merged, will BookKeeperClientWorker-OrderedExecutor thread still call LedgerHandle#sendAddSuccessCallbacks, but will https://github.com/apache/bookkeeper/pull/4171 improve? https://github.com/apache/bookkeeper/pull/4194/commits/f47db3d343c43f583d769428e094978a53e011d1#diff-1d893bb31553b5e1f55c8301d04ae15f38e0d35f531f9dd22475128b7972ddf9R2004 -- 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]
