Ethanlm commented on a change in pull request #3098: [STORM-3481] fix
IllegalArgumentException for ConstraintSolverStrategy
URL: https://github.com/apache/storm/pull/3098#discussion_r311201552
##########
File path: storm-client/src/jvm/org/apache/storm/Config.java
##########
@@ -315,13 +315,6 @@
@IsInteger
@IsPositiveNumber
public static final String TOPOLOGY_RAS_CONSTRAINT_MAX_STATE_SEARCH =
"topology.ras.constraint.max.state.search";
- /**
- * The maximum number of states that will be searched looking for a
solution in the constraint solver strategy.
- * Backward compatibility config value for old topologies
- */
- @IsInteger
- @IsPositiveNumber
- public static final String TOPOLOGY_RAS_CONSTRAINT_MAX_STATE_TRAVERSAL =
"topology.ras.constraint.max.state.traversal";
Review comment:
This was introduced by mistake https://github.com/apache/storm/pull/3056. We
have this config on 0.10 storm in internal mirrors. But this is not an issue
on community Storm codebase
----------------------------------------------------------------
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
