Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/785#issuecomment-146246638 @jerrypeng I keep going back and forth on how we want the annotations to work. Right now they are more or less a one to one translation of what we had before. Admittedly it is a lot cleaner than it was before, but I almost want to combine some of the validators together. Like ``` @IntegerValidator(min=0, nullAllowed=true) public static final String SOME_CONFIG="some.config"; ``` instead of ``` @PositiveIntegerValidator public static final String SOME_CONFIG="some.config"; ``` I'm also not sure about having validator at the end of each annotation. But I don't think we can have a validator named the same as a common class. I want to think this through a bit more.
--- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---