rmetzger commented on pull request #14909: URL: https://github.com/apache/flink/pull/14909#issuecomment-776925910
I rebased this change to the latest master (thus fewer commits are included), and addressed all comments Once you confirm that the `testTaskFailuresAreIgnored()` doesn't make sense, I'll remove it. I once again introduced a MockExecutionGraph for this test. But extracting an interface from the ExecutionGraph is a really involved change that would not be worth it for my needs in this PR. It is easier to introduce a ExecutionGraph interface in a separate change. ---------------------------------------------------------------- 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]
