Github user StefanRRichter commented on a diff in the pull request:
https://github.com/apache/flink/pull/5672#discussion_r174161228
--- Diff:
flink-clients/src/main/java/org/apache/flink/client/cli/Flip6DefaultCLI.java ---
@@ -40,6 +40,7 @@ public Flip6DefaultCLI(Configuration configuration) {
@Override
public boolean isActive(CommandLine commandLine) {
+ this.detachedMode =
commandLine.hasOption(CliFrontendParser.DETACHED_OPTION.getOpt());
--- End diff --
I wonder if there is no better place to initialize this field, instead of
doing it as undocumented side-effect of something that sounds like an unrelated
getter?
---
