[ 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: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org