[STORM-3384] storm set-log-level command throws wrong exception when the topology is not running

• Type: Bug
• Status: Resolved
• Priority: Trivial
• Resolution: Fixed
• Affects Version/s: 1.0.6, 1.1.3, 1.2.2
• Fix Version/s: 1.2.3, 1.1.4
• Labels: pull-request-available
• Assignee: Ethan Li
• Reporter: Ethan Li
• Created: 25/Apr/19 14:18
• Updated: 30/Apr/19 15:28
• Resolved: 26/Apr/19 14:33

Description

https://github.com/apache/storm/blob/1.1.x-branch/storm-core/src/clj/org/apache/storm/command/set_log_level.clj#L31
will throw an exception like the following if the topology is not running

3396 [main] INFO  b.s.c.set-log-level - Sent log config LogConfig(named_logger_level:{ROOT=LogLevel(action:UPDATE, target_log_level:DEBUG, reset_log_level_timeout_secs:30)}) for topology w
Exception in thread "main" java.lang.IllegalArgumentException: No matching field found: IllegalArgumentException for class java.lang.String
	at clojure.lang.Reflector.getInstanceField(Reflector.java:271)
	at clojure.lang.Reflector.invokeNoArgInstanceMember(Reflector.java:315)
	at backtype.storm.command.set_log_level$get_storm_id.invoke(set_log_level.clj:31)
	at backtype.storm.command.set_log_level$_main.doInvoke(set_log_level.clj:75)
	at clojure.lang.RestFn.applyTo(RestFn.java:137)
	at backtype.storm.command.set_log_level.main(Unknown Source)

Issue Links

• links to: GitHub Pull Request #3001
• links to: GitHub Pull Request #3002
• links to: GitHub Pull Request #3003