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

Szilard Nemeth edited comment on YARN-10035 at 12/18/19 6:45 PM:
-----------------------------------------------------------------

Hi [~adam.antal]!
Thanks for this patch.
In WebServices.getApps, you added an if: 

{code:java}
      if (nameQuery != null && nameQuery.equals(appReport.getName())) {
        continue;
      }
{code}

Shouldn't the second part of the statement be? : 

{code:java}
!nameQuery.equals(appReport.getName())
{code}
Actually, you are executing the continue statement if the app name in the query 
matches the name from the appReport, so apps with provided name are skipped.
Am I misunderstanding something?
Otherwise, the patch looks good.



was (Author: snemeth):
Hi [~adam.antal]!
Thanks for this patch.
In WebServices.getApps, you added an if: 

{code:java}

      if (nameQuery != null && nameQuery.equals(appReport.getName())) {
        continue;
      }
{code}

Shouldn't the second part of the statement be? : 

{code:java}
!nameQuery.equals(appReport.getName())
{code}
Actually, you are executing the continue statement if the app name in the query 
matches the name from the appReport, so apps with provided name are skipped.
Am I misunderstanding something?
Otherwise, the patch looks good.


> Add ability to filter the Cluster Applications API request by name
> ------------------------------------------------------------------
>
>                 Key: YARN-10035
>                 URL: https://issues.apache.org/jira/browse/YARN-10035
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: yarn
>    Affects Versions: 3.3.0
>            Reporter: Adam Antal
>            Assignee: Adam Antal
>            Priority: Major
>         Attachments: YARN-10035.001.patch, YARN-10035.002.patch
>
>
> According to the 
> [documentation|https://hadoop.apache.org/docs/r3.2.0/hadoop-yarn/hadoop-yarn-site/ResourceManagerRest.html]
>  we don't support filtering by name in the Cluster Applications API request.
> Usually application tags are a perfect way for tracking applications, but for 
> MR applications the older CLIs usually doesn't support providing app tags, 
> while specifying the name of the job is possible.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org

Reply via email to