apoorvmittal10 commented on PR #19872:
URL: https://github.com/apache/kafka/pull/19872#issuecomment-2934231660

   > @apoorvmittal10 , please let us know if you think what @ji-seung-ryu has 
done is enough, or we require more exploration.
   
   This test without these removed assertions is not doing much, so is the test 
serving any value? Though the doAnswer assertions are not effective as it's on 
separate thread but we can still use AtomicBoolean to figure if condition is 
failing in any other thread. Then in last we can be specific with the error. 
Just a suggestion.


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