[ 
https://issues.apache.org/jira/browse/FLINK-6494?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16057237#comment-16057237
 ] 

ASF GitHub Bot commented on FLINK-6494:
---------------------------------------

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

    https://github.com/apache/flink/pull/4075#discussion_r123197240
  
    --- Diff: 
flink-yarn/src/main/java/org/apache/flink/yarn/configuration/YarnConfigOptions.java
 ---
    @@ -52,7 +52,74 @@
                key("yarn.per-job-cluster.include-user-jar")
                        .defaultValue("ORDER");
     
    +   /**
    +    * The vcores exposed by YARN.
    +    */
    +   public static final ConfigOption<Integer> YARN_VCORES =
    +           key("yarn.containers.vcores")
    +           .defaultValue(Integer.MAX_VALUE);
    --- End diff --
    
    Personally i think we should have methods that allow overriding the default 
for non-String options; I've already opened FLINK-6638 for this.
    
    Let's use -1 as the default instead, it's a more "expected" invalid value i 
suppose.


> Migrate ResourceManager configuration options
> ---------------------------------------------
>
>                 Key: FLINK-6494
>                 URL: https://issues.apache.org/jira/browse/FLINK-6494
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Distributed Coordination, ResourceManager
>            Reporter: Chesnay Schepler
>            Assignee: Fang Yong
>




--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to