Github user jerrypeng commented on the pull request:
https://github.com/apache/storm/pull/658#issuecomment-126573508
Perhaps for the purposes of the config "topology.stats.sample.rate" and
future number based configs we should have a PositiveNumberValidator. There is
already a PositiveIntegerValidator but it only deals with positive integers.
It seems to me that "topology.stats.sample.rate" can be set to some positive
float/double or integer. So I think we should include the
PositiveNumberValidator. I have added a commit that includes the code for the
PositiveNumberValidator. What do you think?
---
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.
---
