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

Weiwei Yang commented on YARN-7863:
-----------------------------------

Hi [~sunilg]

Thanks for the updates. I think v8 patch has addressed most of my concerns. Two 
comments:

PlacementConstraintsUtil.java

canSatisfyNodeConstraintExpresssion: looks like the logic only supports 
affinity to node-attributes, does it support anti-affinity? e.g 
{{targetNotIn(NODE, nodeAttribute("java", "1.8"))}}, can container with such PC 
be allocated to nodes where {{java != 1.8}}? If it is not straightforward to 
support this with {{targetIn}} and {{targetNotIn}}, I think we can add a API 
something like {{targetNodeAttribute(Operator.EQ, "java", "1.8")}} ? For this 
version, we can claim only to support {{EQ}} and {{NE}}. What do you think?

TestPlacementConstraints.java

line 72 and 85: why it is still {{nodeAttribute("java", "java=1.8")}}? The 
value should be "1.8" alone right?

Thanks

 

> 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-YARN-3409.002.patch, 
> YARN-7863-YARN-3409.003.patch, YARN-7863-YARN-3409.004.patch, 
> YARN-7863-YARN-3409.005.patch, YARN-7863-YARN-3409.006.patch, 
> YARN-7863-YARN-3409.007.patch, YARN-7863-YARN-3409.008.patch, 
> 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