dlg99 commented on issue #3466:
URL: https://github.com/apache/bookkeeper/issues/3466#issuecomment-1263923092

   @RaulGracia I am trying to wrap my head around how 
https://github.com/apache/bookkeeper/pull/2794 could result in a missing ledger 
handle.
   
   One thing off with the PR is that it calls blocking lh.close() from the 
callback running on the main threadpool. AFAICT there are no deadlocks because 
whatever else happens runs on the ZK. Zk can be quite slow because of injected 
network errors.
   So here is the proposed change: 
https://github.com/apache/bookkeeper/pull/3513 (waiting for the tests to finish 
up)
   
   Can the lost ledger handle might be caused by the way Pravega handles errors 
in ledger open?


-- 
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]

Reply via email to