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

    https://github.com/apache/flink/pull/4991#discussion_r156717761
  
    --- Diff: 
flink-runtime/src/main/java/org/apache/flink/runtime/clusterframework/types/ResourceProfile.java
 ---
    @@ -61,16 +80,26 @@
         * @param heapMemoryInMB The size of the heap memory, in megabytes.
         * @param directMemoryInMB The size of the direct memory, in megabytes.
         * @param nativeMemoryInMB The size of the native memory, in megabytes.
    +    * @param memoryForInputInMB The size of the memory for input, in 
megabytes.
    +    * @param memoryForOutputInMB The size of the memory for output, in 
megabytes.
         */
        public ResourceProfile(
                        double cpuCores,
                        int heapMemoryInMB,
                        int directMemoryInMB,
    -                   int nativeMemoryInMB) {
    +                   int nativeMemoryInMB,
    +                   int memoryForInputInMB,
    +                   int memoryForOutputInMB,
    --- End diff --
    
    Alright. Can we add this with a separate PR? I would like to not mix 
different things with each other.


---

Reply via email to