Github user sohami commented on a diff in the pull request:
https://github.com/apache/drill/pull/1168#discussion_r174882554
--- Diff: exec/jdbc-all/pom.xml ---
@@ -473,6 +473,8 @@
<exclude>org/yaml/**</exclude>
<exclude>hello/**</exclude>
<exclude>webapps/**</exclude>
+ <exclude>**/org/apache/calcite/avatica/metrics/**</exclude>
+ <exclude>**/org/apache/calcite/avatica/org/**</exclude>
--- End diff --
Would be great to make that change. Other than that, changes looks good to
me---
