[GitHub] [flink] StephanEwen commented on issue #8846: [FLINK-12766][runtime] Dynamically allocate TaskExecutor's managed memory to slots.

2019-07-10 Thread GitBox
StephanEwen commented on issue #8846: [FLINK-12766][runtime] Dynamically allocate TaskExecutor's managed memory to slots. URL: https://github.com/apache/flink/pull/8846#issuecomment-510169482 The code is overall in decent shape. I am a bit unsure, though, about this approach. For

[GitHub] [flink] StephanEwen commented on issue #8846: [FLINK-12766][runtime] Dynamically allocate TaskExecutor's managed memory to slots.

2019-07-09 Thread GitBox
StephanEwen commented on issue #8846: [FLINK-12766][runtime] Dynamically allocate TaskExecutor's managed memory to slots. URL: https://github.com/apache/flink/pull/8846#issuecomment-509729439 I am currently going through this PR. About the changes to the `ResourceProfile`: -