hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Wangda Tan (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (YARN-5716) Add global scheduler interface definition and update CapacityScheduler to use it.
Date Tue, 18 Oct 2016 23:52:58 GMT

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

Wangda Tan updated YARN-5716:
-----------------------------
    Attachment: YARN-5716.003.patch

Thanks [~jianhe] for reviews, 

bq. 1 millisecond ? is this intentional ?
This is to avoid busy loop and pinned on CPU, sleep 1 ms can make CPU usage close to 0%.

bq. getReservedSchedulerKey vs getAllocatedSchedulerKey: looks like the same ?
Yeah they're similiar, since they're not new from this patch, I prefer to address this separately.


bq. Too many data structures which wrapps container related info with similar names: CSAssignment,
ContainerAllocation, ContainerAllocationContext, SchedulerContainer.. could you consolidate
these ?
Agree, but I prefer to address them separately, CSAssignment/ContainerAllocation are not introduced
by this patch. Relation between ContainerAllocationContext and SchedulerContainer is has-more,
so we don't need to consolidate the latter two. 

bq. SchedulingPlacementSet has a bunch unused methods and introduced unused class ResourceRequestUpdateResult,
can we add them when needed ?
That is part of the SchedulingPlacementSet definition, I prefer to keep them here. My next
patch will start using it.

Addressed all other comments. (ver.3)

> Add global scheduler interface definition and update CapacityScheduler to use it.
> ---------------------------------------------------------------------------------
>
>                 Key: YARN-5716
>                 URL: https://issues.apache.org/jira/browse/YARN-5716
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Wangda Tan
>            Assignee: Wangda Tan
>         Attachments: YARN-5716.001.patch, YARN-5716.002.patch, YARN-5716.003.patch
>
>
> Target of this JIRA:
> - Definition of interfaces / objects which will be used by global scheduling, this will
be shared by different schedulers.
> - Modify CapacityScheduler to use it.



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

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


Mime
View raw message