[ https://issues.apache.org/jira/browse/YARN-444?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13631978#comment-13631978 ]
Sandy Ryza commented on YARN-444: --------------------------------- bq. As a logic extension of this, I think we should convert the ExitStatus into an object which holds... What would be the use of the meaning/description? ContainerStatus already contains a diagnostic string that in theory should be holding the same information, right? Uploading a patch that adds the annotations to ContainerExitStatus, removes the references to specific literals, and adds an enum to yarn_protos. > Move special container exit codes from YarnConfiguration to API > --------------------------------------------------------------- > > Key: YARN-444 > URL: https://issues.apache.org/jira/browse/YARN-444 > Project: Hadoop YARN > Issue Type: Sub-task > Components: api, applications/distributed-shell > Affects Versions: 2.0.3-alpha > Reporter: Sandy Ryza > Assignee: Sandy Ryza > Attachments: YARN-444-1.patch, YARN-444.patch > > > YarnConfiguration currently contains the special container exit codes > INVALID_CONTAINER_EXIT_STATUS = -1000, ABORTED_CONTAINER_EXIT_STATUS = -100, > and DISKS_FAILED = -101. > These are not really not really related to configuration, and > YarnConfiguration should not become a place to put miscellaneous constants. > Per discussion on YARN-417, appmaster writers need to be able to provide > special handling for them, so it might make sense to move these to their own > user-facing class. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators For more information on JIRA, see: http://www.atlassian.com/software/jira