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

Benjamin Teke commented on YARN-10548:
--------------------------------------

[~snemeth] [~quapaw] are these comments to be resolved in this item, or a 
followup? As most of the patch is moving around the existing code, mostly 
unchanged, I'm voting for the idea of a global, larger cleanup jira after the 
related patches are merged to avoid any conflict resolution related problems on 
the dependent items.

> Decouple AM runner logic from SLSRunner
> ---------------------------------------
>
>                 Key: YARN-10548
>                 URL: https://issues.apache.org/jira/browse/YARN-10548
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Szilard Nemeth
>            Assignee: Szilard Nemeth
>            Priority: Minor
>         Attachments: YARN-10548.001.patch, YARN-10548.002.patch, 
> YARN-10548.003.patch
>
>
> SLSRunner has too many responsibilities.
>  One of them is to parse the job details from the SLS input formats and 
> launch the AMs and task containers.
>  The AM runner logic could be decoupled.



--
This message was sent by Atlassian Jira
(v8.20.1#820001)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to