p-szucs opened a new pull request, #6499:
URL: https://github.com/apache/hadoop/pull/6499
<!--
Thanks for sending a pull request!
1. If this is your first time, please read our contributor guidelines:
https://cwiki.apache.org/confluence/display/HADOOP/How+To+Contribute
2. Make sure your PR title starts with JIRA issue id, e.g.,
'HADOOP-17799. Your PR title ...'.
-->
### Description of PR
JIRA ticket: https://issues.apache.org/jira/browse/YARN-11041
The changes on this PR are mostly replacing string queuePath occurences to
QueuePath objects.
There are also some minor refactorings recommended by the JIRA ticket, which
are the followings:
AutoCreatedQueueTemplate#setTemplateEntriesForChild method: simplified this
method, and moved some parts of the logic to the QueuePath class
created QueuePrefixes class and moved some static methods there from the
CapacitySchedulerConfiguration class to convert the QueuePath object to various
queue prefix strings that are ultimately keys in the Configuration object
QueuePrefixes#getAutoCreatedQueueObjectTemplateConfPrefix method: AQC
Template prefix was set as a leaf in QueuePath constructor, which seemed hacky,
refactored this method
for CapacitySchedulerConfiguration#getQueues method changed the return type
to list, because all callers had to explicitly do that
### How was this patch tested?
Locally and unit tests.
### For code changes:
- [x] Does the title or this PR starts with the corresponding JIRA issue id
(e.g. 'HADOOP-17799. Your PR title ...')?
- [ ] Object storage: have the integration tests been executed and the
endpoint declared according to the connector-specific documentation?
- [ ] If adding new dependencies to the code, are these dependencies
licensed in a way that is compatible for inclusion under [ASF
2.0](http://www.apache.org/legal/resolved.html#category-a)?
- [ ] If applicable, have you updated the `LICENSE`, `LICENSE-binary`,
`NOTICE-binary` files?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
