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

Konstantinos Karanasos commented on YARN-7669:
----------------------------------------------

Patch looks good to me, thanks [~asuresh].

PS: In the latest names of the {{RejectionReason}} you make the assumption that 
"place" is what the processor does and "schedule" is what the scheduler does 
when trying to commit the resource. In other parts of the code, placement is 
used differently (e.g., {{PlacementManager}} and {{PlacementRule}} -- this is 
what I was referring to in [my 
comment|https://issues.apache.org/jira/browse/YARN-7612?focusedCommentId=16296117&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16296117]
 in YARN-7612). In any case, the javadoc is clear now, so I guess we can rename 
the enums later.

> [API] Introduce interfaces for placement constraint processing
> --------------------------------------------------------------
>
>                 Key: YARN-7669
>                 URL: https://issues.apache.org/jira/browse/YARN-7669
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Arun Suresh
>            Assignee: Arun Suresh
>         Attachments: YARN-7669-YARN-6592.001.patch, 
> YARN-7669-YARN-6592.002.patch, YARN-7669-YARN-6592.003.patch, 
> YARN-7669-YARN-6592.004.patch, YARN-7669-YARN-6592.005.patch
>
>
> As per discussions in YARN-7612. This JIRA will introduce the generic 
> interfaces which will be implemented in YARN-7612



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to