Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/4481
Will merge this now.
---
Github user StefanRRichter commented on the issue:
https://github.com/apache/flink/pull/4481
LGTM ð
---
Github user NicoK commented on the issue:
https://github.com/apache/flink/pull/4481
rebased again - this should be good to go. @StefanRRichter can you continue
with this?
---
Github user NicoK commented on the issue:
https://github.com/apache/flink/pull/4481
FYI: I just rebased this PR onto current `master` to make this mergable and
support further extensions
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user NicoK commented on the issue:
https://github.com/apache/flink/pull/4481
by cherry-picking the commits from #4506, plus some fixes for code which
was changed in the wrong way previously, the failing yarn tests should now be
fixed
---
If your project is set up for it, you
Github user NicoK commented on the issue:
https://github.com/apache/flink/pull/4481
ok, one test fixed, the other is not so simple but maybe @tillrohrmann can
help with it:
Inside `ContaineredTaskManagerParameters#create()`, we calculate the amount
of off-heap space that we
Github user NicoK commented on the issue:
https://github.com/apache/flink/pull/4481
actually, I need to fix the test failures in
`ContaineredTaskManagerParametersTest` and some failure in the
`flink-yarn-tests` first...
---
If your project is set up for it, you can reply to this
