jnturton commented on code in PR #2821: URL: https://github.com/apache/drill/pull/2821#discussion_r1305093910
########## exec/jdbc-all/pom.xml: ########## @@ -388,51 +396,52 @@ <include>*:*</include> </includes> <excludes> - <exclude>io.protostuff:*</exclude> Review Comment: @cgivre All the diff noise here comes about because I decided to sort this list alphabetically in the hopes that contributors will keep it sorted from here on, making checking what's in it that much easier. I can revert the sorting if preferred though. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@drill.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org