davidradl commented on code in PR #25770:
URL: https://github.com/apache/flink/pull/25770#discussion_r1882148465
##########
flink-runtime/src/test/java/org/apache/flink/runtime/scheduler/adaptive/AdaptiveSchedulerTest.java:
##########
@@ -664,7 +740,7 @@ void testNumRestartsMetric() throws Exception {
// wait for the second task submissions
taskManagerGateway.waitForSubmissions(PARALLELISM);
- assertThat(numRestartsMetric.getValue()).isEqualTo(1L);
+ assertThat(numRestartsMetric.getValue()).isEqualTo(0L);
Review Comment:
we seem to have removed the test for numRestartsMetric being incremented. I
think we still need to test that - maybe in another test
--
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
