Myasuka commented on pull request #12529:
URL: https://github.com/apache/flink/pull/12529#issuecomment-641682439


   @StephanEwen Thanks for your review. I realized this problem when I created 
this PR, and I actually try to use [Parameterized test on single method with 
junit5](https://www.baeldung.com/parameterized-tests-junit-5). However, I found 
Flink still use junit4, and then I give up this choice.
   Since you also bring up this problem, I will refactor current tests.
   
   BTW, this PR is for release-1.11, and Yu already give some comments for PR 
targets to master branch at https://github.com/apache/flink/pull/12514 . We can 
put our comments there and then update all related PRs for different branches.


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

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


Reply via email to