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 9: (3 comments) http://gerrit.cloudera.org:8080/#/c/11413/8//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/11413/8//COMMIT_MSG@9 PS8, Line 9: This is necessary so that Impala can override the table owner, instead : of using the inferred username of the connection (which would be : 'impala'). This API is marked as unstable since it's not clear we will : want to support it long term, and there are some open questions about : the fine-grained privileges required to use it. > Fleshing this out for my own understanding: today, without the HMS integrat Correct. http://gerrit.cloudera.org:8080/#/c/11413/8/java/kudu-client/src/test/java/org/apache/kudu/client/TestHiveMetastoreIntegration.java File java/kudu-client/src/test/java/org/apache/kudu/client/TestHiveMetastoreIntegration.java: http://gerrit.cloudera.org:8080/#/c/11413/8/java/kudu-client/src/test/java/org/apache/kudu/client/TestHiveMetastoreIntegration.java@64 PS8, Line 64: > nit: add /*varNames=*/ turns out these overrides weren't necessary. http://gerrit.cloudera.org:8080/#/c/11413/8/src/kudu/master/master.proto File src/kudu/master/master.proto: http://gerrit.cloudera.org:8080/#/c/11413/8/src/kudu/master/master.proto@424 PS8, Line 424: he > nit: with It was correct as 'as', but since it was ambiguous I rephrased it. -- 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: 9 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 Gerrit-Comment-Date: Wed, 17 Oct 2018 21:06:57 +0000 Gerrit-HasComments: Yes
