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
---