KurtYoung commented on a change in pull request #10427: [FLINK-14566] Enable to 
get/set whether an operator uses managed memory
URL: https://github.com/apache/flink/pull/10427#discussion_r354171651
 
 

 ##########
 File path: 
flink-core/src/main/java/org/apache/flink/api/dag/Transformation.java
 ##########
 @@ -136,6 +138,14 @@ public static int getNewNodeId() {
         */
        private ResourceSpec preferredResources = ResourceSpec.DEFAULT;
 
+       /**
+        * This weight indicates how much this transformation relies on managed 
memory, so that
+        * transformation highly relies on managed memory would be able to 
acquire more managed
+        * memory in runtime (linear association). Note that it only works in 
cases of UNKNOWN
+        * resources.
+        */
+       private int managedMemoryWeight = DEFAULT_MANAGED_MEMORY_WEIGHT;
 
 Review comment:
   So what kind of ResourceSpec I should use when some operator doesn't need 
managed memory? First set UNKNOWN resource and then set managed memory to 0 
explicitly?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to