azagrebin commented on issue #9801: [FLINK-13983][runtime] Launch task executor 
with new memory calculation logics
URL: https://github.com/apache/flink/pull/9801#issuecomment-542591930
 
 
    > I would say we keep the scope of this PR as it is, because it's already 
quite large and has been reviewed a lot. We can create a new issue for updating 
ResourceProfile/ResourceSpec under FLIP-49, and get it reviewed independently. 
Since I already have the code, there shouldn't be much extra work. What do you 
think?
   
   Sounds good to me. This change will also simplify [memory manager per slot 
refactoring](https://jira.apache.org/jira/browse/FLINK-14400) where we need 
both limits for the created slot and its memory manager in TM.

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