[GitHub] [hadoop] bteke commented on a diff in pull request #3763: YARN-10850 Fix the application attempt ID query with the correspondin…

[GitBox <gi...@apache.org>]

bteke commented on code in PR #3763:
URL: https://github.com/apache/hadoop/pull/3763#discussion_r848155726


##########
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-client/src/main/java/org/apache/hadoop/yarn/client/api/impl/AHSv2ClientImpl.java:
##########
@@ -137,9 +137,7 @@ public ContainerReport getContainerReport(ContainerId containerId)
     ApplicationId appId = applicationAttemptId.getApplicationId();
     ApplicationReport appReport = getApplicationReport(appId);
     Map<String, String> filters = new HashMap<>();
-    filters.put("infofilters", "SYSTEM_INFO_PARENT_ENTITY eq {\"id\":\"" +
-        applicationAttemptId.toString() +
-        "\",\"type\":\"YARN_APPLICATION_ATTEMPT\"}");
+    filters.put("appattemptId", applicationAttemptId.toString());

Review Comment:
   @szilard-nemeth built the solution proposal, tested it, and it indeed doesn't work, the related ATS endpoint doesn't have a queryparam called [appattemptId](https://github.com/apache/hadoop/blob/18f959d56a6dafd045a54a9d669bb969305947d3/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-timelineservice/src/main/java/org/apache/hadoop/yarn/server/timelineservice/reader/TimelineReaderWebServices.java#L606).



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: common-issues-help@hadoop.apache.org