[GitHub] flink pull request #5463: [FLINK-8475][config][docs] Integrate YARN options

[aljoscha]

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

    https://github.com/apache/flink/pull/5463#discussion_r167607077
  
    --- Diff: 
flink-yarn/src/main/java/org/apache/flink/yarn/configuration/YarnConfigOptions.java
 ---
    @@ -79,14 +87,18 @@
         */
        public static final ConfigOption<String> APPLICATION_ATTEMPTS =
                key("yarn.application-attempts")
    -           .noDefaultValue();
    +           .noDefaultValue()
    --- End diff --
    
    We don't really recommend to leave this at `1` anymore, do we?


---