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

    https://github.com/apache/drill/pull/1189#discussion_r184223843
  
    --- Diff: logical/pom.xml ---
    @@ -85,14 +85,12 @@
         </dependency>
         
         <dependency>
    -      <groupId>com.codahale.metrics</groupId>
    +      <groupId>io.dropwizard.metrics</groupId>
    --- End diff --
    
    No, it is. Thanks for catching this. 
    Also I have noticed that `java-exec` used `io.dropwizard.metrics` as 
transitive from `drill-common`.
    For consistency I have removed  `io.dropwizard.metrics` from 
`drill-memory-base` (it can leverage metrics from `drill-common` too).


---

Reply via email to