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

Arun Suresh commented on YARN-5457:
-----------------------------------

I propose the following changes :

* Move {{OpportunisticContainerAllocator}} to yarn-server-common
** Fix the constructor to just pass in the appropriate TokenSecretManager 
instead of the NodeStatusUpdater.
* Move the {{DistributedScheduler}} class to yarn-server-common and probably 
move all the app specific bookkeeping to the SchedulerNode.
* Rename {{DistributedSchedulingAMService}} to 
{{OpportunisticSchedulingAMService}}
** modify the {{getServer}} method to enable the new protocol only if 
DistributedScheduling is enabled.
** modify the allocate method to intercept all OPPORTUNISTIC requests and use 
the {{OpportunisticContainerAllocator}} to allocate and create the containers.


> Refactor DistributedScheduling framework to pull out common functionality
> -------------------------------------------------------------------------
>
>                 Key: YARN-5457
>                 URL: https://issues.apache.org/jira/browse/YARN-5457
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>            Reporter: Arun Suresh
>
> Opening this JIRA to track the some refactoring missed in YARN-5113:



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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