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

    https://github.com/apache/flink/pull/5448#discussion_r191147488
  
    --- Diff: 
flink-core/src/main/java/org/apache/flink/configuration/TaskManagerOptions.java 
---
    @@ -32,10 +32,22 @@
        //  General TaskManager Options
        // 
------------------------------------------------------------------------
     
    +   /**
    +    * JVM heap size for the TaskManagers with memory size.
    +    */
    +   public static final ConfigOption<String> TASK_MANAGER_HEAP_MEMORY =
    +                   key("taskmanager.heap.size")
    +                   .defaultValue("1024m")
    +                   .withDescription("JVM heap size for the TaskManagers, 
which are the parallel workers of" +
    +                                   " the system. On YARN setups, this 
value is automatically configured to the size of the TaskManager's" +
    +                                   " YARN container, minus a certain 
tolerance value.");
    +
        /**
         * JVM heap size (in megabytes) for the TaskManagers.
    +    *
    +    * @deprecated use {@link #TASK_MANAGER_HEAP_MEMORY}
         */
    -   public static final ConfigOption<Integer> TASK_MANAGER_HEAP_MEMORY =
    +   public static final ConfigOption<Integer> TASK_MANAGER_HEAP_MEMORY_MB =
    --- End diff --
    
    Add `@Deprecated`


---

Reply via email to