[ https://issues.apache.org/jira/browse/STORM-3734?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Ethan Li updated STORM-3734: ---------------------------- Description: The IntegerValidator allows the non-integer object, like Double(1.0). https://github.com/apache/storm/blob/7bef73a6faa14558ef254efe74cbe4bfef81c2e2/storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java#L404-L415 It can be reproduced by {code:java} IntegerValidator validator = new IntegerValidator(); validator.validateInteger("test", 1.0); {code} More details at https://github.com/apache/storm/pull/3365#issuecomment-754775896 was: The IntegerValidator allows the non-integer object, like Double(1.0). https://github.com/apache/storm/blob/7bef73a6faa14558ef254efe74cbe4bfef81c2e2/storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java#L404-L415 It can be reproduced by {code:java} IntegerValidator validator = new IntegerValidator(); validator.validateInteger("test", 1.0); {code} > IntegerValidator doesn't force the object type to be Integer > ------------------------------------------------------------ > > Key: STORM-3734 > URL: https://issues.apache.org/jira/browse/STORM-3734 > Project: Apache Storm > Issue Type: Bug > Reporter: Ethan Li > Priority: Major > > The IntegerValidator allows the non-integer object, like Double(1.0). > https://github.com/apache/storm/blob/7bef73a6faa14558ef254efe74cbe4bfef81c2e2/storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java#L404-L415 > It can be reproduced by > {code:java} > IntegerValidator validator = new IntegerValidator(); > validator.validateInteger("test", 1.0); > {code} > More details at > https://github.com/apache/storm/pull/3365#issuecomment-754775896 -- This message was sent by Atlassian Jira (v8.3.4#803005)