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

Zhijie Shen commented on YARN-947:
----------------------------------

bq. BTW yarnapplicationattemptstate is already part of YARN-978.

Yes, I know that. [~jeffgx619], are you fine if I takes 
YarnApplicationAttemptState part from your patch of YARN-978? This is because 
we want the branch is always compilable.

bq. I think we should use the same states In AHS as we have in RM and we don't 
think we should use others states then RM states as thats what we show users to 
RM UI/CLI to avoid confusion.

It's not the other states. Actually web UI and CLI is showing 
YarnApplicationState instead or RMAppState, as YarnApplicationState is a user 
oriented object (it is a proto). That's why we propose 
YarnApplicationAttemptState in YARN-978, and we also need YarnContainerState as 
well.

> Defining the history data classes for the implementation of the 
> reading/writing interface
> -----------------------------------------------------------------------------------------
>
>                 Key: YARN-947
>                 URL: https://issues.apache.org/jira/browse/YARN-947
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Zhijie Shen
>            Assignee: Zhijie Shen
>             Fix For: YARN-321
>
>         Attachments: YARN-947.1.patch, YARN-947.2.patch, YARN-947.3.patch, 
> YARN-947.4.patch, YARN-947.5.patch
>
>
> We need to define the history data classes have the exact fields to be 
> stored. Therefore, all the implementations don't need to have the duplicate 
> logic to exact the required information from RMApp, RMAppAttempt and 
> RMContainer.
> We use protobuf to define these classes, such that they can be ser/des 
> to/from bytes, which are easier for persistence.



--
This message was sent by Atlassian JIRA
(v6.1#6144)

Reply via email to