Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/5672#discussion_r175713468
--- Diff:
flink-clients/src/main/java/org/apache/flink/client/cli/ProgramOptions.java ---
@@ -56,8 +55,6 @@
private final boolean stdoutLogging;
- private final boolean detachedMode;
--- End diff --
I think we should not remove these fields. The `ProgramOptions` represent
general options for running a program. The execution behaviour is in my opinion
one of these options.
---
