Github user srdo commented on a diff in the pull request:
https://github.com/apache/storm/pull/2675#discussion_r189456044
--- Diff: external/storm-jms/pom.xml ---
@@ -62,8 +62,7 @@
<!-- Active MQ -->
<dependency>
<groupId>org.apache.activemq</groupId>
- <artifactId>activemq-core</artifactId>
- <version>5.5.1</version>
+ <artifactId>activemq-all</artifactId>
--- End diff --
Nit: Not sure we need everything in this jar, maybe activemq-client and
activemq-broker would be enough?---
