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

ASF GitHub Bot commented on YARN-8898:
--------------------------------------

slfan1989 commented on code in PR #5645:
URL: https://github.com/apache/hadoop/pull/5645#discussion_r1191852103


##########
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-common/src/main/java/org/apache/hadoop/yarn/server/uam/UnmanagedAMPoolManager.java:
##########
@@ -188,14 +189,15 @@ public String createAndRegisterNewUAM(
   public Token<AMRMTokenIdentifier> launchUAM(String uamId, Configuration conf,
       ApplicationId appId, String queueName, String submitter,

Review Comment:
   Thank you for your help in reviewing the code! I will first fix the java doc 
involved in this pr, and then I will submit a separate pr to fix all the java 
docs related to yarn federation (different modules may require 1-2 pr).





> Fix FederationInterceptor#allocate to set application priority in 
> allocateResponse
> ----------------------------------------------------------------------------------
>
>                 Key: YARN-8898
>                 URL: https://issues.apache.org/jira/browse/YARN-8898
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Bibin Chundatt
>            Assignee: Shilun Fan
>            Priority: Major
>              Labels: pull-request-available
>         Attachments: YARN-8898.wip.patch
>
>
> In case of FederationInterceptor#mergeAllocateResponses skips 
> application_priority in response returned



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to