eolivelli commented on issue #2879: URL: https://github.com/apache/bookkeeper/issues/2879#issuecomment-961915952
`The Journal is a BookieCriticalThread ` in fact I am surprised that we are catching that exception. Probably this is due to the fact that we are supporting multiple journals. But I believe that we should terminate the Bookie as soon as possible. `in the BookieImpl run() method with a check to see if any journal thread has exited, this will then lead to a controlled shutdown.` I agree with this proposal. @aloyszhang would you like to send to send a patch ? you can use PowerMock in order to simulate the IO failure for instance -- 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]
