[FLINK-15846] TaskExecutorProcessUtils should return config strs with mb as unit for better readability

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Invalid
• Component/s: Runtime / Configuration
• Labels: pull-request-available
• Assignee: Unassigned
• Reporter: Steven Zhen Wu
• Created: 02/Feb/20 21:51
• Updated: 03/Feb/20 05:42
• Resolved: 03/Feb/20 05:41

Description

With FLIP-49 being released in 1.10. The config strs returned `TaskExecutorResourceUtils` are using bytes as unit which are very difficult to read when inspecting logs. It is hard to believe anyone will ever configure memory using bytes as unit. mostly likely either mega-bytes or giga-bytes.

e.g. from logs.

BASH_JAVA_UTILS_EXEC_RESULT:-D taskmanager.memory.framework.off-heap.size=2147483648b -D taskmanager.memory.network.max=163840000b -D taskmanager.memory.network.min=163840000b -D taskmanager.memory.framework.heap.size=134217728b -D taskmanager.memory.managed.size=2616406869b -D taskmanager.cpu.cores=2.0 -D taskmanager.memory.task.heap.size=1479068832b -D taskmanager.memory.task.off-heap.size=0b

-Xmx1613286560
-Xms1613286560
-XX:MaxDirectMemorySize=2311323648
-XX:MaxMetaspaceSize=536870912

Issue Links

• links to: GitHub Pull Request #10994