[
https://issues.apache.org/jira/browse/YARN-2009?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Sunil G updated YARN-2009:
--------------------------
Attachment: YARN-2009.0004.patch
Thank [~leftnoteasy] for the comments. Updating new patch addressing the
comments.
bq.f. orderedApps is invented to avoid sorting, correct? If you need to sort it
again, we don't need it.
orderedApps is a list of TempAppPerPartition and comparator is designed based
on FiFo. So initial list is created from the PriorityQueue, and we removed each
entry from top to get the high priority app. {{orderedApps}} has just collected
those apps in order, and then later re-sorted to get apps in reverse order.
{{tq.getApps}} return FiCaSchedulerApp, but we need TempAppPerPartition. So i
think {{orderedApps}} is needed for the logic. Did i miss something?
> Priority support for preemption in ProportionalCapacityPreemptionPolicy
> -----------------------------------------------------------------------
>
> Key: YARN-2009
> URL: https://issues.apache.org/jira/browse/YARN-2009
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: capacityscheduler
> Reporter: Devaraj K
> Assignee: Sunil G
> Attachments: YARN-2009.0001.patch, YARN-2009.0002.patch,
> YARN-2009.0003.patch, YARN-2009.0004.patch
>
>
> While preempting containers based on the queue ideal assignment, we may need
> to consider preempting the low priority application containers first.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
