[
https://issues.apache.org/jira/browse/YARN-2404?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Tsuyoshi OZAWA updated YARN-2404:
---------------------------------
Attachment: YARN-2404.3.patch
Jian, thanks for your review. Updated a patch to address your comments:
1. Made FileSystemRMStateStore#getAppDir accepts ApplicationId.
2. LOG.error("Error storing app: " + appId, e) in StoreAppTransition.
3. Removed needless instance creation in RMStateStore#removeApplication.
4. Moved convertCredentailsFromByteBuffer from TestRMRestart to
ApplicationAttemptStateData.
> Remove ApplicationAttemptState and ApplicationState class in RMStateStore
> class
> --------------------------------------------------------------------------------
>
> Key: YARN-2404
> URL: https://issues.apache.org/jira/browse/YARN-2404
> Project: Hadoop YARN
> Issue Type: Sub-task
> Reporter: Jian He
> Assignee: Tsuyoshi OZAWA
> Attachments: YARN-2404.1.patch, YARN-2404.2.patch, YARN-2404.3.patch
>
>
> We can remove ApplicationState and ApplicationAttemptState class in
> RMStateStore, given that we already have ApplicationStateData and
> ApplicationAttemptStateData records. we may just replace ApplicationState
> with ApplicationStateData, similarly for ApplicationAttemptState.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
