hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Arun Suresh (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-7920) Cleanup configuration of PlacementConstraints
Date Wed, 14 Feb 2018 07:49:00 GMT

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

Arun Suresh commented on YARN-7920:

Thanks for working on this [~leftnoteasy]

Couple of comments:
* the {{ amsProcessingChain.init(rmContext, null);}} call should be in the {{initializeProcessingChain}}
* With regard to the {{SchedulerPlacementProcessor}}, we are assuming that if enabled, then
placement constraints CANNOT be specified via the registerAM call.. Technically, you can still
specify constraints in the register call - the schedulingRequest just overrides it.
* I agree with [~kkaranasos], we should make the value of the handler something the user should
make sense of. Given that the major difference between the two approaches are the fact that
the scheduler still handles for request in priority order and the processor tries to optimize
for placement, ignoring priority, maybe we should call it "priority-optimized" and "placement-optimized"
? Thoughts ?

> Cleanup configuration of PlacementConstraints
> ---------------------------------------------
>                 Key: YARN-7920
>                 URL: https://issues.apache.org/jira/browse/YARN-7920
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Wangda Tan
>            Assignee: Wangda Tan
>            Priority: Blocker
>         Attachments: YARN-7920.001.patch, YARN-7920.002.patch, YARN-7920.003.patch
> Currently it is very confusing to have the two configs in two different files (yarn-site.xml
and capacity-scheduler.xml). 
> Maybe a better approach is: we can delete the scheduling-request.allowed in CS, and update
placement-constraints configs in yarn-site.xml a bit: 
> - Remove placement-constraints.enabled, and add a new placement-constraints.handler,
by default is none, and other acceptable values are a. external-processor (since algorithm
is too generic to me), b. scheduler. 
> - And add a new PlacementProcessor just to pass SchedulingRequest to scheduler without
any modifications.

This message was sent by Atlassian JIRA

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

View raw message