Attila Bukor has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/19026 )

Change subject: KUDU-3401 Unable to query Kudu tables from Hive with Kudu HMS 
Integration enabled
......................................................................


Patch Set 4:

(3 comments)

http://gerrit.cloudera.org:8080/#/c/19026/3//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/19026/3//COMMIT_MSG@25
PS3, Line 25: Manually tested on a
            : seperate cluster, confirmed the missing data is sent and written 
to the
            : Hive Database.
> Yes, the queries work fine, didn't run into any other errors.
Can you add this to the commit msg?


http://gerrit.cloudera.org:8080/#/c/19026/4//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/19026/4//COMMIT_MSG@7
PS4, Line 7: KUDU-3401 Unable to query Kudu tables from Hive with Kudu HMS
nit: shorten the subject line. Commit subjects shouldn't copy the JIRA subject. 
It should be concise, and explain what the commit will do when applied (e.g. 
"KUDU-1234 Fix something")


http://gerrit.cloudera.org:8080/#/c/19026/4//COMMIT_MSG@18
PS4, Line 18: Hms
nit: capitalize



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia1b53b55005e2899d8575b0fb7250351d914afb4
Gerrit-Change-Number: 19026
Gerrit-PatchSet: 4
Gerrit-Owner: Khazar Mammadli <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Khazar Mammadli <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Comment-Date: Tue, 27 Sep 2022 14:55:46 +0000
Gerrit-HasComments: Yes

Reply via email to