David Ribeiro Alves has posted comments on this change.

Change subject: Fix CTAS syntax, improve formatting of CREATE TABLE statements
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/2523/1/docs/kudu_impala_integration.adoc
File docs/kudu_impala_integration.adoc:

Line 371: TBLPROPERTIES(
> OK, so I extrapolated from a cloudera bug report where the TBLPROPERTIES wa
no, our syntax for CTAS is the same, the difference is that after the table 
definition a ctas has a select statement that a normal create table doesn't. in 
any case if you look at all the other non ctas examples in this page (some of 
which you touch in this patch) they all have the tblproperties at the end.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic9e3b3c196caeaa6bb7132ad02e6528e3bb342cd
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Misty Stanley-Jones <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Misty Stanley-Jones <[email protected]>
Gerrit-HasComments: Yes

Reply via email to