C0urante commented on PR #14186: URL: https://github.com/apache/kafka/pull/14186#issuecomment-1674202980
@divijvaidya given that this touches on the strict stubbing feature that you've helped introduce to our tests, would you be interested in reviewing? @mumrah @ijuma based on discussion on [KAFKA-4594](https://issues.apache.org/jira/browse/KAFKA-4594) and https://github.com/apache/kafka/pull/2695 it seems like the original intent behind the two-task approach (i.e., running `unitTest integrationTest` instead of just `test`) was to get faster feedback from our CI builds, but that this was a minor concern compared to the goal of the ticket to just be able to run unit tests in isolation locally. Given that it's a bit like finding a needle in a haystack now to sift through in-progress CI results and this PR doesn't compromise the original goal laid out in Jira, this change seems safe enough to make. However, in case there's context I'm missing or you feel differently, I wanted to give you a chance to weigh in on this change. -- 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
