[GitHub] flink issue #5448: [FLINK-6469] Configure Memory Sizes with units

[dawidwys]

Github user dawidwys commented on the issue:

    https://github.com/apache/flink/pull/5448
  
    Hi @yanghua 
    I think those changes look overall good. The biggest concern I had with 
handling `MANAGED_MEMORY_SIZE` as I think if used with old style configuration 
file, the resulting value will differ.


---