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.
---