Github user tillrohrmann commented on a diff in the pull request:

    https://github.com/apache/flink/pull/5431#discussion_r167852786
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/entrypoint/ClusterEntrypoint.java
 ---
    @@ -63,6 +89,10 @@
      */
     public abstract class ClusterEntrypoint implements FatalErrorHandler {
     
    +   public static final ConfigOption<String> EXECUTION_MODE = ConfigOptions
    +           .key("cluster.execution-mode")
    --- End diff --
    
    There are other configuration parameters as well like 
`local.number-taskmanager`. In the old Yarn code, we used environment variables 
to pass certain settings. I think the configuration is a better place. However, 
I agree that this might be a bit confusing. I'll add the prefix `internal` to 
the key.


---

Reply via email to