[
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)