kishorvpatil commented on a change in pull request #3215: Storm3585 - New
compact Constraint config including maxCoLocationCnt
URL: https://github.com/apache/storm/pull/3215#discussion_r387757114
##########
File path: storm-client/src/jvm/org/apache/storm/Config.java
##########
@@ -304,11 +307,13 @@
// an error will be thrown by nimbus on topology submission and not by the
client prior to submitting
// the topology.
public static final String TOPOLOGY_SCHEDULER_STRATEGY =
"topology.scheduler.strategy";
+
/**
- * Declare scheduling constraints for a topology used by the constraint
solver strategy. A List of pairs (also a list) of components
- * that cannot coexist in the same worker.
+ * Declare scheduling constraints for a topology used by the constraint
solver strategy. The format can be either
+ * old style (validated by ListOfListOfStringValidator.class or the newer
style, which is a list of specific type of
+ * Maps (validated by RasConstraintsTypeValidator.class). The value must
be in one or the other format.
Review comment:
Some examples of old style and new style configurations might help.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
