K0K0V0K commented on code in PR #3618:
URL: https://github.com/apache/hadoop/pull/3618#discussion_r1210232008
##########
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/TestCapacitySchedulerNewQueueAutoCreation.java:
##########
@@ -1264,7 +1272,9 @@ public void testAutoCreateInvalidParent() throws
Exception {
protected AbstractLeafQueue createQueue(String queuePath) throws
YarnException,
IOException {
- return autoQueueHandler.createQueue(new QueuePath(queuePath));
+ AbstractLeafQueue result = autoQueueHandler.createQueue(new
QueuePath(queuePath));
+ cs = CapacitySchedulerTestUtilities.setupCapacityScheduler(mockRM, 1200);
Review Comment:
I thought i did that for reason cause it is not a local variable, but i
removed that and seems like the tests still not failing even the ones whos
inherited from this class, so i will fix it
--
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]
