Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11733 )
Change subject: Add missing test dependency for Hive tests ...................................................................... Patch Set 1: Tracked down a bit more info on what's going on here. The HMS has an 'optional' dependency on hadoop-mapreduce-client-core, even though it 100% absolutely requires it to be on the classpath. Gradle never pulls in optional dependencies (except, it appears, at compile time). Somehow the branch-2 version of hive that we run against fulfills the dependency transitively through HBase, however it appears this isn't the case for other versions of Hive (again, notably CDH 6.x-SNAPSHOT). Since it's definitely a dependency of the HMS, and it's marked optional, I think it's appropriate to also include it as a test dependency here. -- To view, visit http://gerrit.cloudera.org:8080/11733 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I33187880247b3fe930ff521b0671345646e0fbf4 Gerrit-Change-Number: 11733 Gerrit-PatchSet: 1 Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Fri, 19 Oct 2018 00:42:04 +0000 Gerrit-HasComments: No
