[STORM-3734] IntegerValidator doesn't force the object type to be Integer

• Type: Bug
• Status: Open
• Priority: Major
• Resolution: Unresolved
• Assignee: Unassigned
• Reporter: Ethan Li
• Created: 05/Jan/21 17:26
• Updated: 05/Jan/21 17:28

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

        IntegerValidator validator = new IntegerValidator();
        validator.validateInteger("test", 1.0);

More details at https://github.com/apache/storm/pull/3365#issuecomment-754775896

Issue Links

• relates to: STORM-3727 SUPERVISOR_SLOTS_PORTS could be list of Longs