Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/11273 )
Change subject: [hms] Add Hcatalog jars to the HMS classpath ...................................................................... Patch Set 2: (1 comment) http://gerrit.cloudera.org:8080/#/c/11273/2/src/kudu/hms/mini_hms.cc File src/kudu/hms/mini_hms.cc: http://gerrit.cloudera.org:8080/#/c/11273/2/src/kudu/hms/mini_hms.cc@119 PS2, Line 119: string aux_jars = Substitute("$0/hms-plugin.jar,$1/hcatalog/share/hcatalog/*", bin_dir, hive_home); Looks like you need to wrap this to make lint happy. -- To view, visit http://gerrit.cloudera.org:8080/11273 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ifed85fffdabb5af19f22e343843e012b009888a0 Gerrit-Change-Number: 11273 Gerrit-PatchSet: 2 Gerrit-Owner: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Hao Hao <hao....@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Tue, 21 Aug 2018 04:08:28 +0000 Gerrit-HasComments: Yes