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

    https://github.com/apache/drill/pull/1189#discussion_r184066086
  
    --- Diff: pom.xml ---
    @@ -1333,6 +1353,12 @@
               </exclusion>
             </exclusions>
           </dependency>
    +      <dependency>
    --- End diff --
    
    I thought that Drill uses it. But I have checked and found that there is no 
direct using of this kind of metrics. Therefore 4 dependencies blocks can be 
removed. 
    
    And now `com.yammer.metrics` is a transitive dependency for Drill and is a 
intra-project dependency for `hbase-server`, and `kafka_2.11`. Since the 
dependency doesn't conflict with `io.dropwizard` it would be good just to 
control the version of it in `dependency-management` block to avoid issues in 
future.


---

Reply via email to