[
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: [email protected]
For additional commands, e-mail: [email protected]
