faaronzheng commented on issue #10073: [FLINK-14560] [runtime] The value of 
taskmanager.memory.size in flink-conf.yaml is set to zero will cause 
taskmanager not to work
URL: https://github.com/apache/flink/pull/10073#issuecomment-549206556
 
 
   > Thanks for opening this PR @faaronzheng. It took me a while to understand 
the underlying problem. Instead of defining `0` to be the "take the fraction" 
value, I would suggest to remove the default value from 
`LEGACY_MANAGED_MEMORY_SIZE`. If the user should set this value to `0`, then it 
should fail as it represents an invalid system configuration.
   
    LEGACY_MANAGED_MEMORY_SIZE is already removed the default value, I think it 
is okay to remove the default value from MANAGED_MEMORY_SIZE in release-1.9 and 
branch
   
   

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