dengziming opened a new pull request, #12165:
URL: https://github.com/apache/kafka/pull/12165

   *More detailed description of your change*
   Before #11969, We will throw an `ExecutionException(KafkaStorageException)` 
in `BrokerServer.startup`, and the outside ExecutionException is removed in 
finally block.
   After #11969, We will throw a 
`RuntimeException(ExecutionException(KafkaStorageException))`, so this test is 
constantly failing since the Exception type is not consistent.
   
   To fix this, we just need to invoke `getCause` twice to remove 
RuntimeException and ExecutionException.
   
   *Summary of testing strategy (including rationale)*
   This test is no longer failing.
   
   ### Committer Checklist (excluded from commit message)
   - [ ] Verify design and implementation 
   - [ ] Verify test coverage and CI build status
   - [ ] Verify documentation (including upgrade notes)
   


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