K0K0V0K commented on code in PR #8641:
URL: https://github.com/apache/hadoop/pull/8641#discussion_r3731090902


##########
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:
   Yeah, i mean if someone modify the exception message in 
CapacitySchedulerConfigValidator, they have to modify here also, would be nice 
if the string would be stored in one place



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