Dan Burkert has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/11413 )

Change subject: HMS integration: provide Java API to override owner during 
table create
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/11413/4/java/kudu-client/build.gradle
File java/kudu-client/build.gradle:

http://gerrit.cloudera.org:8080/#/c/11413/4/java/kudu-client/build.gradle@42
PS4, Line 42:     // Avro conflicts with kudu-flume's Avro dependency, and 
isn't necessary for the HMS client.
            :     exclude group: "org.apache.avro", module: "avro"
> Don't we need the same exclusion in pom.xml too?
I haven't had any issues running the maven build/tests locally.  Do we have a 
policy of keeping them consistent anyway?



--
To view, visit http://gerrit.cloudera.org:8080/11413
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I21069d81a4b54e55c399933b69789bf83bd58de0
Gerrit-Change-Number: 11413
Gerrit-PatchSet: 4
Gerrit-Owner: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <andrew.w...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Tue, 11 Sep 2018 22:49:58 +0000
Gerrit-HasComments: Yes

Reply via email to