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

Steve Loughran commented on MAPREDUCE-4014:
-------------------------------------------

I've looked at the code and written the tests, the {{CompositeService}} seems 
good apart from MAPREDUCE-4016, where I need to verify that sharing of 
configurations is not a feature people actually wanted. If MAPREDUCE-4015 was 
included -a {{YarnException}} subclass specifically for illegal state changes, 
{{CompositeService.init()}} could be extended to relay this exception up, 
rather than wrap it. 

The patch I'm submitting has merged in the {{AbstractService}} changes to aid 
making service stop operations robust regardless of service state; these were 
taken from MAPREDUCE-3502. When committed, and MAPREDUCE-4015/MAPREDUCE-4016 
resolved, this should complete the yarn-common changes and then specific 
services can be reviewed. 
                
> Make yarn-common services robust
> --------------------------------
>
>                 Key: MAPREDUCE-4014
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-4014
>             Project: Hadoop Map/Reduce
>          Issue Type: Sub-task
>          Components: mrv2
>    Affects Versions: 0.23.1, 0.24.0
>            Reporter: Steve Loughran
>            Assignee: Steve Loughran
>              Labels: yarn
>         Attachments: MAPREDUCE-4014.patch
>
>
> Review the yarn common services ({{CompositeService}}, 
> {{AbstractLivelinessMonitor}} and make their service startup _and especially 
> shutdown_ more robust against out-of-lifecycle invocation and partially 
> complete initialization.
> Write tests for these where possible. 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to