Matthias Pohl
Matthias Pohl
> this thing from PR description is not clear to me > if we temporarily enable it, what is the long term plan here? I was referring to debug commit...
I rebased the branch to most-recent `master`. But it won't make much of a difference: The failed tests in CI are unrelated to the changes of this PR.
I created FLINK-34921 to cover the one test failure which should be unrelated to this PR's change. The second test failure is related to FLINK-34643
[CI with AdaptiveScheduler](https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=58505&view=results) looks good. Only FLINK-34643 is failing. I'm gonna revert the CI change to make the PR ready to be merged.
@flinkbot run azure
@xintongsong Do you remember what the actual purpose of this test was? Am I missing something or can we actually remove this test because it might be unstable on slower...
Could we finalize this one? :thinking:
I added the changes I proposed. That should be good enough from my end, if you're ok with those changes (just to bring the PR closer to being merged and...
> I added the changes I proposed. That should be good enough from my end, if you're ok with those changes (just to bring the PR closer to being merged...
@flinkbot run azure