[GitHub] activemq-artemis pull request #2081: ARTEMIS-1783 add Guava dep in artemis-c...

[clebertsuconic]

Github user clebertsuconic commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/2081#discussion_r187389048
  
    --- Diff: artemis-core-client/pom.xml ---
    @@ -118,6 +118,10 @@
              <groupId>io.netty</groupId>
              <artifactId>netty-common</artifactId>
           </dependency>
    +      <dependency>
    +         <groupId>com.google.guava</groupId>
    --- End diff --
    
    why you need that? core-client doesn't use it directly.. only commons is 
using guava. I thought we should keep it that way.


---