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

Tao Yang commented on YARN-9490:
--------------------------------

Hi, [~cheersyang], could you please help to give contributor permission to 
[~zyb] and review this issue?

Findbugs warnings are unrelated to this patch(will be resolved in YARN-9495).

Regarding the test case, there is no existing case to test fields in 
ApplicationResourceUsageReport, this bug is obvious and the fix is clean 
enough, I think it's fine for the patch without a test case.

> applicationresourceusagereport return wrong number of reserved containers
> -------------------------------------------------------------------------
>
>                 Key: YARN-9490
>                 URL: https://issues.apache.org/jira/browse/YARN-9490
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: resourcemanager
>    Affects Versions: 3.3.0
>            Reporter: yanbing zhang
>            Priority: Minor
>             Fix For: 3.3.0
>
>         Attachments: YARN-9490.002.patch, YARN-9490.patch, 
> YARN-9490.patch1.patch
>
>
> when getting an ApplicationResourceUsageReport instance from the class of 
> SchedulerApplicationAttempt, I found the input constructor 
> parameter(reservedContainers.size()) is wrong.  because the type of this 
> variable is Map<SchedulerRequestKey, Map<NodeId, RMContainer>>, so 
> "reservedContainer.size()" is not the number of containers, but the number of 
> SchedulerRequestKey.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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

Reply via email to