[
https://issues.apache.org/jira/browse/HADOOP-17165?focusedWorklogId=479375&page=com.atlassian.jira.plugin.system.issuetabpanels:worklog-tabpanel#worklog-479375
]
ASF GitHub Bot logged work on HADOOP-17165:
-------------------------------------------
Author: ASF GitHub Bot
Created on: 05/Sep/20 23:25
Start Date: 05/Sep/20 23:25
Worklog Time Spent: 10m
Work Description: sunchao commented on a change in pull request #2240:
URL: https://github.com/apache/hadoop/pull/2240#discussion_r483999578
##########
File path:
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/ipc/DecayRpcScheduler.java
##########
@@ -483,6 +501,12 @@ private void recomputeScheduleCache() {
for (Map.Entry<Object, List<AtomicLong>> entry : callCosts.entrySet()) {
Object id = entry.getKey();
+ // The priority for service users is always 0
+ if (isServiceUser((String)id)) {
Review comment:
Gotcha. Thanks @tasanuma . This looks most good for me. One minor nit is
that the logic for handling service users now is separated into two places.
Another way perhaps is to add this in `computePriorityLevel` which is called by
`recomputeScheduleCache` and `cacheOrComputePriorityLevel`. We need to add an
extra parameter for this method though. What do you think?
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
Issue Time Tracking
-------------------
Worklog Id: (was: 479375)
Time Spent: 1h (was: 50m)
> Implement service-user feature in DecayRPCScheduler
> ---------------------------------------------------
>
> Key: HADOOP-17165
> URL: https://issues.apache.org/jira/browse/HADOOP-17165
> Project: Hadoop Common
> Issue Type: New Feature
> Reporter: Takanobu Asanuma
> Assignee: Takanobu Asanuma
> Priority: Major
> Labels: pull-request-available
> Attachments: HADOOP-17165.001.patch, HADOOP-17165.002.patch,
> after.png, before.png
>
> Time Spent: 1h
> Remaining Estimate: 0h
>
> In our cluster, we want to use FairCallQueue to limit heavy users, but not
> want to restrict certain users who are submitting important requests. This
> jira proposes to implement the service-user feature that the user is always
> scheduled high-priority queue.
> According to HADOOP-9640, the initial concept of FCQ has this feature, but
> not implemented finally.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
