[ 
https://issues.apache.org/jira/browse/YARN-4628?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Bibin A Chundatt updated YARN-4628:
-----------------------------------
    Attachment: 0003-YARN-4628.patch

[~sunilg]

Thanks for review comment.

By default if we are sorting on prority will be done in descending order  since 
{{ascendingSort = false}} by default . (Higher priority first) .

{noformat}
    if (ascendingSort) {
      Collections.sort(appsInfo, comparator);
    } else {
      Collections.sort(appsInfo, Collections.reverseOrder(comparator));
    }
{noformat}
Also we do have option for reversing in in top cli. 

[~vvasudev]
Handled latest review comments too. Attaching lated patch. Please do review

> TopCli to support Application priority 
> ---------------------------------------
>
>                 Key: YARN-4628
>                 URL: https://issues.apache.org/jira/browse/YARN-4628
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Bibin A Chundatt
>            Assignee: Bibin A Chundatt
>            Priority: Minor
>         Attachments: 0001-YARN-4628.patch, 0002-YARN-4628.patch, 
> 0003-YARN-4628.patch
>
>
> In Topcli support  Apppriority 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to