[ https://issues.apache.org/jira/browse/YARN-10943?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17430358#comment-17430358 ]
Szilard Nemeth commented on YARN-10943: --------------------------------------- The bulk of this change will be implemented with YARN-10904. Anyway, let's keep this open and double-check what remains after YARN-10904 gets merged. > AbstractCSQueue: Create separate class for encapsulating Min / Max Resource > --------------------------------------------------------------------------- > > Key: YARN-10943 > URL: https://issues.apache.org/jira/browse/YARN-10943 > Project: Hadoop YARN > Issue Type: Sub-task > Reporter: Szilard Nemeth > Assignee: Szilard Nemeth > Priority: Minor > > There are certain methods where min and max Resources are used in tandem. > Some examples of these kind of methods: > - getMinimumAbsoluteResource / getMaximumAbsoluteResource > - *updateConfigurableResourceLimits:* > - It invokes setConfiguredMinResource / setConfiguredMaxResource on > QueueResourceQuotas. That object could define a simple method that receives > the MinMaxResource alone. > - Validator methods are also receiving min/max resources as separate > parameters, which could be tied together. > - updateEffectiveResources: It performs operations with effective min/max > resources. > Alternatively, 2 classes could be created: > - One for EffectiveMinMaxResource > - And another for AbsoluteMinMaxResource -- This message was sent by Atlassian Jira (v8.3.4#803005) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org