You are viewing a plain text version of this content. The canonical link for it is here.

Posted to yarn-issues@hadoop.apache.org by "Tsuyoshi OZAWA (JIRA)" <ji...@apache.org> on 2014/09/03 06:16:53 UTC

[jira] [Updated] (YARN-2404) Remove ApplicationAttemptState and ApplicationState class in RMStateStore class

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