divijvaidya commented on PR #12285: URL: https://github.com/apache/kafka/pull/12285#issuecomment-1153118745
Thanks you submitting this change @clolov. This would greatly help us in moving towards getting rid of JUnit4 completely. 1\ To make the code review easier, could we please tackle the `Run IntelliJ's Optimize Imports and streams:spotlessApply on the streams module.` in a separate review? I would drastically decrease the number of files changed and help us reviewing the code in a better manner. 2\ I agree with (and appreciate) your incremental approach towards this migration. We can handle the tests using parameterised separately. 3\ It would be helpful if you can document the high level changes required for the migration from Junit4 to Junit5 in the description of this PR. -- 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
