GJL commented on issue #10858: [FLINK-15582] Change LegacySchedulerBatchSchedulingTest to DefaultSchedulerBatchSchedulingTest URL: https://github.com/apache/flink/pull/10858#issuecomment-593404444 I would feel better running the tests for both legacy and `DefaultScheduler`. I currently don't see why, for example, `ExecutionGraphPartitionReleaseTest` should not be run using the legacy scheduler. >For the other tests which test the scheduling, including this one, if you feels it's better to keep the tests for legacy scheduler before we made the decision to remove it, we can postpone rewriting those tests to avoid some unnecessary duplicate work. ok > For this PR, since I have such a PR already, I can revert it to the last version, namely having it > testing both the legacy scheduler and new scheduler. ok
---------------------------------------------------------------- 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: [email protected] With regards, Apache Git Services
