[STORM-3581] Change log level to info to show the config classes being used for validation

• Type: Improvement
• Status: Closed
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 2.0.0, 2.1.0
• Fix Version/s: 2.2.0, 2.1.1
• Labels: pull-request-available
• Assignee: Ethan Li
• Reporter: Ethan Li
• Created: 13/Feb/20 17:12
• Updated: 01/Jul/20 14:08
• Resolved: 14/Feb/20 21:28

Description

https://github.com/apache/storm/blob/master/storm-client/src/jvm/org/apache/storm/validation/ConfigValidation.java#L82

This is trivial but since it's caused some confusion, it's better to have it in the log as INFO instead of DEBUG

LOG.debug("Will use {} for validation", ret);

 

Because the classes being used for validation depends on whether the following file is in the classpath or not

 

https://github.com/apache/storm/blob/master/storm-server/src/main/resources/META-INF/services/org.apache.storm.validation.Validated

Issue Links

• links to: GitHub Pull Request #3209