dekimsey opened a new pull request #3423: URL: https://github.com/apache/storm/pull/3423
## What is the purpose of the change The `component` value is a string, not a number, see [Monitor.java](https://github.com/apache/storm/blob/master/storm-core/src/jvm/org/apache/storm/utils/Monitor.java#L156). Attempting to use a number throws a stacktrace like such: ``` ~/apache-storm-2.3.0/bin/storm monitor -m wordGenerator production-topology topology component parallelism stream time-diff ms emitted throughput (Kt/s) Available components for production-topology : ------------------ __acker wordGenerator intermediateRanker counter finalRanker ------------------ Exception in thread "main" java.lang.IllegalArgumentException: component: wordGeneratotor not found at org.apache.storm.utils.Monitor.metrics(Monitor.java:128) at org.apache.storm.utils.Monitor.metrics(Monitor.java:83) at org.apache.storm.command.Monitor$1.run(Monitor.java:53) at org.apache.storm.utils.NimbusClient.withConfiguredClient(NimbusClient.java:128) at org.apache.storm.utils.NimbusClient.withConfiguredClient(NimbusClient.java:117) at org.apache.storm.command.Monitor.main(Monitor.java:50) ``` ## How was the change tested The monitor command now works. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@storm.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
