Github user revans2 commented on the pull request:
https://github.com/apache/incubator-storm/pull/132#issuecomment-45217514
So with this new code if someone somehow sets a config that has a key of 5
it will essentially adjust the TIMEOUT-CHECK-SECS value? That seems ugly to
me. I was thinking of using conjure but because TIMEOUT-CHECK-SECS is not a
function I see that we cannot stub it. Perhaps we should just make a true key
for it in Config.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---