soarez commented on PR #14369:
URL: https://github.com/apache/kafka/pull/14369#issuecomment-1810719729

   > avoids calling exit(1) in junit tests, which will kill Jenkins dead (even 
after 3 decades of Java, we don't have the technology to intercept exit() in 
unit testrs >:( )
   
   @cmccabe I'm confused, I thought that was the whole point of 
`kafka.utils.Exit`
   
https://github.com/apache/kafka/blob/trunk/core/src/main/scala/kafka/utils/Exit.scala#L21-L24
   
https://github.com/apache/kafka/blob/00e5803cd3af89011254e734232308618403309d/clients/src/test/java/org/apache/kafka/common/utils/ExitTest.java
   
   The `FaultHandler` concept in `QuorumController` seems great, but I'm not 
sure how it fits here: afaict the fault is always fatal and we can intercept it 
in tests - what am I missing?


-- 
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: jira-unsubscr...@kafka.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to