Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/8316 )
Change subject: kudu-hive: add provided scope dependency on hadoop-mapreduce-client-common ...................................................................... Patch Set 2: (1 comment) Fix looks good and I know I'm late to the party, just one small comment. http://gerrit.cloudera.org:8080/#/c/8316/2/java/kudu-hive/pom.xml File java/kudu-hive/pom.xml: http://gerrit.cloudera.org:8080/#/c/8316/2/java/kudu-hive/pom.xml@67 PS2, Line 67: <dependency> Would be nice to add a comment here justifying its existence. -- To view, visit http://gerrit.cloudera.org:8080/8316 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I62d62ae31ebd82a2fd39030bfa6201aef6ff82ff Gerrit-Change-Number: 8316 Gerrit-PatchSet: 2 Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Wed, 18 Oct 2017 21:15:35 +0000 Gerrit-HasComments: Yes
