Github user NicoK commented on the issue:
https://github.com/apache/flink/pull/4939
1) yes, I have tested the changes on a YARN cluster as described in the PR
comment (before the changes on the unit tests)
2) regarding the `pom.xml` changes: I actually took those from
991af3652479f85f732cbbade46bed7df1c5d819 and
36b663f4561458408ee68902a1db6a5cd539e1c2 and can only guess why @StephanEwen
and @zentol used this way except for what you proposed. Maybe they can clarify.---
