[
https://issues.apache.org/jira/browse/YARN-7920?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16363645#comment-16363645
]
Wangda Tan commented on YARN-7920:
----------------------------------
Thanks [~asuresh],
Addressed:
{quote} * the \{{ amsProcessingChain.init(rmContext, null);}} call should be in
the {{initializeProcessingChain}} method.
* 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.{quote}
For:
{quote}maybe we should call it "priority-optimized" and "placement-optimized" ?
Thoughts ?
{quote}
I would prefer not, the processor handler could improve priority support and
the scheduler handler could improve placement support. More likely, the two
handler will be merged in the future into the same one and user doesn't need to
choose. Let's keep the name open so we don't have to find a different name once
the functionalities of each handler get improved.
Attached ver.4, please review.
> 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, YARN-7920.004.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
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
