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

Arun Suresh commented on YARN-4829:
-----------------------------------

The patch looks mostly good. Thanks [~vvasudev]

Couplo minor nits:
* Maybe rename *Mi, Ti, Pi* to *Me, Te, Pe* or maybe replace eveything with *b 
(Kb, Mb..) to signify binary ?
* Can we have a test case that converts between a binary to non-binary (K to 
Ki) for eg. ? 

> Add support for binary units
> ----------------------------
>
>                 Key: YARN-4829
>                 URL: https://issues.apache.org/jira/browse/YARN-4829
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: nodemanager, resourcemanager
>            Reporter: Varun Vasudev
>            Assignee: Varun Vasudev
>         Attachments: YARN-4829-YARN-3926.001.patch
>
>
> The units conversion util should have support for binary units.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to