Github user zentol commented on a diff in the pull request: https://github.com/apache/flink/pull/6140#discussion_r199760506 --- Diff: flink-clients/src/main/java/org/apache/flink/client/cli/CliFrontend.java --- @@ -166,6 +166,10 @@ public Configuration getConfiguration() { return copiedConfiguration; } + public static Options getCustomCommandLineOptions() { --- End diff -- it should be possible for this getter to be non-static. Then the changes to the `customCommandLineOptions` field are unnecessary, and the in `FlinkShell` replace `CliFrontend.getCustomCommandLineOptions()` with `frontend.getCustomCommandLineOptions`
---