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

Reply via email to