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

Wangda Tan commented on YARN-6593:
----------------------------------

Thanks [~kkaranasos] for your explanations. 

Yeah I think you're correct, for the current PB, we cannot do extend easily on 
the PB side. However we can still improve Java API:

For example, PlacementConstraints: Inside the PlacementConstraintProto, we can 
add CompoundType / Children / DelayCriteria / Everything in the 
SimplePlacementConstraintProto. And a field to indicate if it is a 
CompoundConstraint / (Simple)TargetConstraint / (Simple)CardinalityConstraint.

With this, we can at least make Java API can be very clear, same as what we 
defined in design doc, and Java side takes responsibility to translate 
user-facing API and PB. Please share your thoughts, +[~asuresh]. 

bq. My question was more about how do we validate constraints 
+1 to have a validator in common layer so client side can do validation as 
well. I think validation has two phases: 1) Client side static validation 2) 
Server side dynamic validation, #2 will look at cluster state and check things 
like if constraints added to cluster or not, etc.

> [API] Introduce Placement Constraint object
> -------------------------------------------
>
>                 Key: YARN-6593
>                 URL: https://issues.apache.org/jira/browse/YARN-6593
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Konstantinos Karanasos
>            Assignee: Konstantinos Karanasos
>         Attachments: YARN-6593.001.patch
>
>
> This JIRA introduces an object for defining placement constraints.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
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