Matthew Jacobs has posted comments on this change.

Change subject: IMPALA-5546: Allow creating unpartitioned Kudu tables
......................................................................


Patch Set 1:

Nice! I like the warning. How about also a ctas test case?

I think it'd also be useful to move some of the smaller tables in alltypes_kudu 
and tpch_kudu to be unpartitioned. a) because there's no reason for the tiny 
tables to use the overhead and b) probably more realistic for customer 
scenarios. Fine with me if you wanna do that in a separate commit.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I281f173dbec1484eb13434d53ea581a0f245358a
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Thomas Tauber-Marshall <[email protected]>
Gerrit-Reviewer: Matthew Jacobs <[email protected]>
Gerrit-HasComments: No

Reply via email to