Github user dawidwys commented on the issue:
https://github.com/apache/flink/pull/5448
Hi @yanghua,
I am afraid Stephan won't be able to reply any time soon. I would suggest
to
- add the default unit to the parse method of `MemorySize` and use MB for
`MANAGED_MEMORY_SIZE`.
- change the return value of getMebiBytes() to int or have a
getMebiBytesAsInt() method that uses a MathUtils.checkedDownCast() to avoid
unnoticed overflow errors, as Stephan commented
- change the default value of `MANAGED_MEMORY_SIZE` to 0, as suggested by
@zentol
After that I think this PR will be ready to be merged.
---
