[
https://issues.apache.org/jira/browse/YARN-8002?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16399777#comment-16399777
]
Weiwei Yang commented on YARN-8002:
-----------------------------------
Thanks [~leftnoteasy]
bq. Changes in SingleConstraintAppPlacementAllocato...
Make sense, I have removed the change in this patch. Will track in YARN-8015.
bq. TargetApplications: remove unused methods
Done
bq. aggregateAllocationTagsByApps/aggregateAllocationTagsByRac --- The two
methods can be merged.
Good suggestion, I've merged them to one method now. About the null check, I
prefer to keep it in case in future this method is used in some where else.
Just to prevent a potential NPE.
bq. getNamespaceScope ... remove those
OK, I have removed those and replaced by explicit check on enum value.
bq. I suggest to add a method like throwExceptionIfNamespaceTypeNotSupported so
we don't need to change all the places in the future.
That might not be necessary. There are two checks, one in
{{PlacementConstraintsUtil}} and the other in
{{SingleConstraintAppPlacementAllocator}}. I will removed the first one once
YARN-8013 is done. And removed the 2nd one once YARN-8015 is done. But please
let me know if you insist on this, I can add a method in
{{PlacementConstraintsUtil}} and call it in both places.
Thanks
> Support NOT_SELF and ALL namespace types for allocation tag
> -----------------------------------------------------------
>
> Key: YARN-8002
> URL: https://issues.apache.org/jira/browse/YARN-8002
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: resourcemanager
> Reporter: Weiwei Yang
> Assignee: Weiwei Yang
> Priority: Major
> Attachments: YARN-8002.001.patch, YARN-8002.002.patch,
> YARN-8002.003.patch
>
>
> This is a continua task after YARN-7972, YARN-7972 adds support to specify
> tags with namespace SELF and APP_ID, like following
> * self/<tag>
> * app-id/<appid>/<tag>
> this task is to track the work to support 2 of remaining namespace types
> *NOT_SELF* & *ALL* (we'll support app-label later),
> * not-self/<tag>
> * all/<tag>
> this will require a bit refactoring in {{AllocationTagsManager}} as it needs
> to do some proper aggregation on tags for multiple apps.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
