Github user yanghua commented on the issue:
https://github.com/apache/flink/pull/5448
hi @dawidwys thanks for your review suggestion, I have refactored the PR
code except the `MANAGED_MEMORY_SIZE `. The problem you concerned is exists,
the key is suitable for this PR, and it seems we should introduce a new key and
mark this as **@deprecated**, otherwise, We could not avoid user using old
config value without unit.
But it is just a possibility, we can highlight a **Note** message and give
a guidance.
At worst, user config it as **megabytes**, actually it means **bytes""
(`1024 * 1024` difference) , when starting TaskManager, it would cause failed
or show disagreement with Flink web UI. Then use would recheck this
configuration item.
What's your opinion? @StephanEwen
---
