Adar Dembo 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"
> I haven't had any issues running the maven build/tests locally.  Do we have
Until we remove the Maven build entirely, we should continue to ensure it's 
working (i.e. you can run 'mvn install', the modules compile, and the tests 
pass). Weird that you didn't need the exclusion for Maven to keep working 
though.

On a related note, Grant has been itching to remove Maven but wants to run 
through a release or two using Gradle to shake out any remaining bugs.



--
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 23:06:24 +0000
Gerrit-HasComments: Yes

Reply via email to