[
https://issues.apache.org/jira/browse/YARN-10486?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17233298#comment-17233298
]
Peter Bacsko commented on YARN-10486:
-------------------------------------
Thanks [~bteke] let's proceed with approach #2.
I made several changes in patch #5:
* Renamed variables
* Renamed the property to relax-capacity-validation
* Added unit tests to TestParentQueue.java
* Fixed validations in ParentQueue.java
Note that there are two places in ParentQueue.java where you have to switch
between normal and relaxed validation. The previous patch was incorrect. The
"if" conditions might not be completely obvious, let me know if it's confusing
I'll and try to rewrite it.
Another thing is that we can't just let the queues to have arbitrary capacity.
It would be good but the capacity values propagate towards the top (root) and
eventually somewhere along the line it will fail. So basically you'd need to
set the new property to "true" for a given hierarchy, which is really not
convenient. So the sum of capacities must be either 0% or 100%. But it's still
more lenient than the current code.
> FS-CS converter: handle case when weight=0
> ------------------------------------------
>
> Key: YARN-10486
> URL: https://issues.apache.org/jira/browse/YARN-10486
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: yarn
> Reporter: Peter Bacsko
> Assignee: Peter Bacsko
> Priority: Major
> Labels: fs2cs
> Attachments: YARN-10486-001.patch, YARN-10486-002.patch,
> YARN-10486-003-approach2.patch, YARN-10486-004-approach2.patch,
> YARN-10486-005.patch
>
>
> We can encounter an ArithmeticException if there is a single or multiple
> queues under a parent with zero weight.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
