Github user tillrohrmann commented on the issue:
https://github.com/apache/flink/pull/4921
Thanks for the review @zentol. Yes you're right that this PR has failing
test cases. I wasn't aware that the `ParameterTool` is so messy (getters with
side effects, storing default values for future gets, counting which parameters
have been accessed, sending this whole thing over to the cluster) and that the
`RequiredParameters` change the `ParameterTool` directly. I'm currently
thinking about letting the `RequiredParameters` create a new `ParameterTool` if
they have to change a value. Ideally, I would like to remove
`RequiredParameters` and do not send the `ParameterTool` to the cluster.
---
