Hean-Chhinling commented on code in PR #8641:
URL: https://github.com/apache/hadoop/pull/8641#discussion_r3740517096


##########
hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/test/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/capacity/TestCapacitySchedulerConfigValidator.java:
##########
@@ -735,4 +736,96 @@ private CapacitySchedulerConfiguration 
setupCSConfiguration(YarnConfiguration co
 
     return csConf;
   }
+
+  private static final String ACCESSIBLE_LABELS_CAPACITY_ERROR =

Review Comment:
   If you modify the error message then you have to update the unit-test that 
check for message too :)
   To me, I do not see anything wrong with this and usually this is how it works



-- 
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]

Reply via email to