Jiabao-Sun commented on code in PR #23914:
URL: https://github.com/apache/flink/pull/23914#discussion_r1425242859


##########
flink-table/flink-table-planner/src/test/java/org/apache/flink/table/planner/runtime/utils/StreamAbstractTestBase.java:
##########
@@ -22,10 +22,14 @@
 import org.apache.flink.test.junit5.MiniClusterExtension;
 
 import org.junit.jupiter.api.extension.RegisterExtension;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 /** Base class for unit tests that run multiple tests and want to reuse the 
same Flink cluster. */
 public class StreamAbstractTestBase {

Review Comment:
   I made some attempts to put `MiniClusterExtension` in `StreamingTestBase`, 
but it violated the rules of Arch Unit Test (the scala code could not be well 
recognized).
   
   
https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=55472&view=logs&j=0c940707-2659-5648-cbe6-a1ad63045f0a&t=075c2716-8010-5565-fe08-3c4bb45824a4&l=11485



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

Reply via email to