This is an automated email from the ASF dual-hosted git repository. fanrui pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/flink.git
commit 8a04ae03b9f4f29ae906b13f5c80fa9b6cbf1ccd Author: Rui Fan <[email protected]> AuthorDate: Wed Jan 10 17:16:16 2024 +0800 [FLINK-32852][hotfix] DefaultSchedulerTest should use the LOG instead of TestLogger of junit4 --- .../apache/flink/runtime/scheduler/DefaultSchedulerTest.java | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/flink-runtime/src/test/java/org/apache/flink/runtime/scheduler/DefaultSchedulerTest.java b/flink-runtime/src/test/java/org/apache/flink/runtime/scheduler/DefaultSchedulerTest.java index 78eff53d220..db95e29486a 100644 --- a/flink-runtime/src/test/java/org/apache/flink/runtime/scheduler/DefaultSchedulerTest.java +++ b/flink-runtime/src/test/java/org/apache/flink/runtime/scheduler/DefaultSchedulerTest.java @@ -101,7 +101,6 @@ import org.apache.flink.runtime.util.ResourceCounter; import org.apache.flink.util.ExecutorUtils; import org.apache.flink.util.FlinkException; import org.apache.flink.util.Preconditions; -import org.apache.flink.util.TestLogger; import org.apache.flink.util.concurrent.ManuallyTriggeredScheduledExecutor; import org.apache.flink.util.concurrent.ScheduledExecutor; @@ -111,6 +110,7 @@ import org.junit.jupiter.api.AfterEach; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; import org.slf4j.Logger; +import org.slf4j.LoggerFactory; import java.util.ArrayList; import java.util.Arrays; @@ -145,7 +145,9 @@ import static org.assertj.core.api.Assertions.assertThat; import static org.assertj.core.api.Assertions.assertThatThrownBy; /** Tests for {@link DefaultScheduler}. */ -public class DefaultSchedulerTest extends TestLogger { +public class DefaultSchedulerTest { + + private static final Logger LOG = LoggerFactory.getLogger(DefaultSchedulerTest.class); private static final int TIMEOUT_MS = 1000; @@ -1785,7 +1787,7 @@ public class DefaultSchedulerTest extends TestLogger { } }, executorService, - log); + LOG); } finally { executorService.shutdownNow(); } @@ -2115,7 +2117,7 @@ public class DefaultSchedulerTest extends TestLogger { executor, scheduledExecutorService, taskRestartExecutor) - .setLogger(log) + .setLogger(LOG) .setJobMasterConfiguration(configuration) .setSchedulingStrategyFactory(new PipelinedRegionSchedulingStrategy.Factory()) .setFailoverStrategyFactory(new RestartPipelinedRegionFailoverStrategy.Factory())
