You are viewing a plain text version of this content. The canonical link for it is here.
Posted to yarn-issues@hadoop.apache.org by "Konstantinos Karanasos (JIRA)" <ji...@apache.org> on 2017/06/05 23:04:12 UTC
[jira] [Comment Edited] (YARN-6670) Add separate NM overallocation
thresholds for cpu and memory
[ https://issues.apache.org/jira/browse/YARN-6670?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16037812#comment-16037812 ]
Konstantinos Karanasos edited comment on YARN-6670 at 6/5/17 11:03 PM:
-----------------------------------------------------------------------
Thanks for the patch, [~haibochen].
A couple of comments:
* The parameter names that contain both "overallocation" and "allocation" seem a bit redundant. I'd suggest to use either the simpler "overallocation.memory.threshold" or the more precise "overallocation.memory-utilization-threshold".
* Using 0 as a threshold to disable overallocation is somehow counter-intuitive (as in "allow containers to use 0% of the resources"). How about we use -1 to make clear it is disabled?
* The preemption threshold is expected to always be higher than the allocation threshold, right? Shall we add a check for that?
was (Author: kkaranasos):
Thanks for the patch, [~haibochen].
A couple of comments:
* The parameter names that contain both "overallocation" and "allocation" seem a bit redundant. I'd suggest to use either the simpler "overallocation.memory.threshold" or to more precise "overallocation.memory-utilization-threshold".
* Using 0 as a threshold to disable overallocation is somehow counter-intuitive (as in "allow containers to use 0% of the resources"). How about we use -1 to make clear it is disabled?
* The preemption threshold is expected to always be higher than the allocation threshold, right? Shall we add a check for that?
> Add separate NM overallocation thresholds for cpu and memory
> ------------------------------------------------------------
>
> Key: YARN-6670
> URL: https://issues.apache.org/jira/browse/YARN-6670
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: nodemanager
> Affects Versions: 3.0.0-alpha3
> Reporter: Haibo Chen
> Assignee: Haibo Chen
> Attachments: YARN-6670-YARN-1011.00.patch, YARN-6670-YARN-1011.01.patch
>
>
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: yarn-issues-help@hadoop.apache.org