[ 
https://issues.apache.org/jira/browse/FLINK-20258?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17239654#comment-17239654
 ] 

Matthias commented on FLINK-20258:
----------------------------------

I forwarded the memory-related parameter values using bytes for precision 
reasons (and because it is done in the same way for the TaskManager). 
[~vthinkxie] and I discussed having the conversion being done in the frontend 
vs the backend. We decided to do the conversion into human-readable units in 
the UI instead of using {{MemorySize#toHumanReadableString}} in the backend. I 
wanted to document in the comment above that I verified [~vthinkxie]'s 
requirement that the used unit provided by the {{/jobmanager/config}} endpoint 
is stable.

> Configured memory sizes on the JM metrics page should be displayed with 
> proper units.
> -------------------------------------------------------------------------------------
>
>                 Key: FLINK-20258
>                 URL: https://issues.apache.org/jira/browse/FLINK-20258
>             Project: Flink
>          Issue Type: Improvement
>          Components: Runtime / Web Frontend
>            Reporter: Xintong Song
>            Assignee: Yadong Xie
>            Priority: Critical
>              Labels: pull-request-available
>             Fix For: 1.12.0
>
>         Attachments: screenshot.png
>
>
> Configured memory sizes for JM are displayed in bytes. It would be better to 
> use proper units here, same as what we do for TM.
>   !screenshot.png!



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to