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

Zhijie Shen commented on YARN-978:
----------------------------------

Sorry for the breaking message above.

The patch is almost good, but still requires to address the following issue:
* Update the name accordingly the suggestion in YARN-1123
{code}
+message ApplicationAttemptReportProto {
+  optional ApplicationAttemptIdProto applicationAttemptId = 1;
+  optional string host = 2;
+  optional int32 rpc_port = 3;
+  optional string trackingUrl = 4;
+  optional string diagnostics = 5 [default = "N/A"];
+  optional YarnApplicationAttemptStateProto yarn_application_attempt_state = 6;
+  optional ContainerIdProto amContainerId = 7;
+}
{code}

> [YARN-321] Adding ApplicationAttemptReport and Protobuf implementation
> ----------------------------------------------------------------------
>
>                 Key: YARN-978
>                 URL: https://issues.apache.org/jira/browse/YARN-978
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Mayank Bansal
>            Assignee: Mayank Bansal
>             Fix For: YARN-321
>
>         Attachments: YARN-978-1.patch, YARN-978.2.patch, YARN-978.3.patch, 
> YARN-978.4.patch, YARN-978.5.patch, YARN-978.6.patch, YARN-978.7.patch, 
> YARN-978.8.patch, YARN-978.9.patch
>
>
> We dont have ApplicationAttemptReport and Protobuf implementation.
> Adding that.
> Thanks,
> Mayank



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

Reply via email to