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

Arun Suresh commented on YARN-7613:
-----------------------------------

Some more comments:

* Maybe we add the iterator functionality into the BatchedRequest class, i.e 
expose an iterator() method - in which case we wont need the 
SchedulerRequestWrapper.
* Let perform all initialization in the Processor itself: You get the iterator 
type from the configuration in the Processor, and since the Processor is what 
creates the BatchedRequests in the first place, we wont even need to send the 
iterator type down to the Algortihm.
* Lets rename BasicPLacementAlgorithm to DefaultPlacementAlgorithm and remove 
the SamplePlacementAlgorithm class.

> Implement Planning algorithms for rich placement
> ------------------------------------------------
>
>                 Key: YARN-7613
>                 URL: https://issues.apache.org/jira/browse/YARN-7613
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Arun Suresh
>            Assignee: Panagiotis Garefalakis
>         Attachments: YARN-7613-YARN-6592.001.patch, YARN-7613.wip.patch
>
>




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