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