ruanhang1993 commented on a change in pull request #17556: URL: https://github.com/apache/flink/pull/17556#discussion_r747269792
########## File path: flink-runtime/src/test/java/org/apache/flink/runtime/taskmanager/TaskCancelAsyncProducerConsumerITCase.java ########## @@ -58,7 +60,8 @@ import static org.junit.Assert.assertNotNull; import static org.junit.Assert.assertTrue; -public class TaskCancelAsyncProducerConsumerITCase extends TestLogger { +@ExtendWith({TestLoggerExtension.class}) Review comment: Yes, but I want this PR to focus on the job about changing rules to extensions. I change this test to show how to use `AllCallbackWrapper`. And I will raise other PRs to migrate modules to JUnit5. -- 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: issues-unsubscr...@flink.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org