Tamas Mate has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15703 )

Change subject: IMPALA-9640 [DOC]: Document Impala support for Kudu VARCHAR type
......................................................................


Patch Set 1:

(1 comment)

Hi Kristine,

Thank you for the change, just one small nit in the commit message, other than 
that LGTM.

http://gerrit.cloudera.org:8080/#/c/15703/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/15703/1//COMMIT_MSG@9
PS1, Line 9: Removed VARCHAR from the list of unsupported types in "Kudu 
considerations" in the shared common file.
nit: The body of the commit message should have 72 or fewer characters if 
possible.



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

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I61ad6982c35a009b15a2a082692f118a0fbcee65
Gerrit-Change-Number: 15703
Gerrit-PatchSet: 1
Gerrit-Owner: Kristine Hahn <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Tamas Mate <[email protected]>
Gerrit-Comment-Date: Wed, 22 Apr 2020 08:28:40 +0000
Gerrit-HasComments: Yes

Reply via email to