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

Naganarasimha G R commented on YARN-7863:
-----------------------------------------

[~sunilg],


Few queries before going into the detail ...
1. who is doing the validation of the resource request?
2. If there are any open resource request for specified attribute, do we allow 
removing of that attribute from Attribute Manager ?
3. Confused little with the logic (may be earlier code to yours), i was always 
of the opinion that the partition is the first to be compared and then with tag 
or attribute here it seems to first check for the allocation tag and then for 
partition and then attribute. should it not be like under a given parition 
check any nodes having allocation tags and/or attributes satisfying the user's 
request?


Some comments :
1. PlacementConstraintsUtil, ln no 145-156 : Scheduler node needs to be having 
map of NodeAttributeKey -> NodeAttribute value to make it O(1), as well this 
will more useful when we go other types and operator based comparison happens. 
Or to consider for current scenario atleast it would be better to override 
NodeAttribute's equals method.

 

 

> Modify placement constraints to support node attributes
> -------------------------------------------------------
>
>                 Key: YARN-7863
>                 URL: https://issues.apache.org/jira/browse/YARN-7863
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Sunil Govindan
>            Assignee: Sunil Govindan
>            Priority: Major
>         Attachments: YARN-7863.v0.patch
>
>
> This Jira will track to *Modify existing placement constraints to support 
> node attributes.*



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to