[
https://issues.apache.org/jira/browse/YARN-4844?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15261187#comment-15261187
]
Hitesh Shah commented on YARN-4844:
-----------------------------------
bq. Per my understanding, changing from int to long won't affect downstream
project a lot, it's an error which can be captured by compiler directly. And
getMemory/getVCores should not be used intensively by downstream project. For
example, MR uses only ~20 times of getMemory()/VCores for non-testing code.
Which can be easily fixed.
If you are going to force downstream apps to change, I dont understand why you
are not forcing them to do this in the first 3.0.0 release? What benefit does
this provide anyone by delaying it to some later 3.x.y release? It just means
that you have do the production stability verification of upstream apps all
over again.
> Upgrade fields of o.a.h.y.api.records.Resource from int32 to int64
> ------------------------------------------------------------------
>
> Key: YARN-4844
> URL: https://issues.apache.org/jira/browse/YARN-4844
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: api
> Reporter: Wangda Tan
> Assignee: Wangda Tan
> Priority: Blocker
> Attachments: YARN-4844.1.patch, YARN-4844.2.patch, YARN-4844.3.patch
>
>
> We use int32 for memory now, if a cluster has 10k nodes, each node has 210G
> memory, we will get a negative total cluster memory.
> And another case that easier overflows int32 is: we added all pending
> resources of running apps to cluster's total pending resources. If a
> problematic app requires too much resources (let's say 1M+ containers, each
> of them has 3G containers), int32 will be not enough.
> Even if we can cap each app's pending request, we cannot handle the case that
> there're many running apps, each of them has capped but still significant
> numbers of pending resources.
> So we may possibly need to upgrade int32 memory field (could include v-cores
> as well) to int64 to avoid integer overflow.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
