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

Zoltan Siegl commented on YARN-9358:
------------------------------------

[~pbacsko] thank you for the review, I have changed those accordingly to "fair 
share of the queue" and "The passed resource object". These sound even better. 
:) 

> Add javadoc to new methods introduced in FSQueueMetrics with YARN-9322
> ----------------------------------------------------------------------
>
>                 Key: YARN-9358
>                 URL: https://issues.apache.org/jira/browse/YARN-9358
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Szilard Nemeth
>            Assignee: Zoltan Siegl
>            Priority: Major
>         Attachments: YARN-9358.001.patch, YARN-9358.002.patch, 
> YARN-9358.003.patch, YARN-9358.004.patch
>
>
> This is a follow-up for YARN-9322, covering javadoc changes as discussed with 
> [~templedf] earlier.
> As discussed with Daniel, we need to add javadoc for the new methods 
> introduced with YARN-9322 and also for the modified methods. 
> The javadoc should refer to the fact that Resource Types are also included in 
> the Resource object in case of get/set as well.
> The methods are: 
> 1. getFairShare / setFairShare
> 2. getSteadyFairShare / setSteadyFairShare
> 3. getMinShare / setMinShare
> 4. getMaxShare / setMaxShare
> 5. getMaxAMShare / setMaxAMShare
> 6. getAMResourceUsage / setAMResourceUsage
> Moreover, a javadoc could be added to the constructor of FSQueueMetrics as 
> well.



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

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

Reply via email to