impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Casey Ching (Code Review)" <>
Subject [Impala-CR](cdh5-trunk) Simplify creating external Kudu tables and add DROP DATABASE CASCADE
Date Sat, 23 Apr 2016 00:58:19 GMT
Casey Ching has posted comments on this change.

Change subject: Simplify creating external Kudu tables and add DROP DATABASE CASCADE

Patch Set 3:

Commit Message:

Line 23: KUDU as a file format
> Yeah, I'm wondering if you've chatted with them about what they're planning
Just sent an email to Lenni/you about this.
File fe/src/main/java/com/cloudera/impala/analysis/

Line 298: rowFormat_ = RowFormat.DEFAULT_ROW_FORMAT;
> Yeah, keeping it null seems fine as long as whatever code that reads it lat
Done. The tests seem to pass.
File fe/src/main/java/com/cloudera/impala/service/

Line 1129: For
         :     // now Impala will assume that any table not in its cache also doesn't require
         :     // use of a DdlDelegate.
> Have you chatted with the Hive team about their kudu integration plans? We 
Not yet, just asked about this in the email.
File testdata/workloads/functional-query/queries/QueryTest/kudu-show-create.test:

Line 9: )
> Ah yes, but that's not the only thing: also that this comes back with the s
File tests/

Line 264: conn
> If you don't mind pulling test infra stuff into a separate review I think i

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic141102818b6dad3016181b179a14024d0ff709d
Gerrit-PatchSet: 3
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Casey Ching <>
Gerrit-Reviewer: Casey Ching <>
Gerrit-Reviewer: Matthew Jacobs <>
Gerrit-HasComments: Yes

View raw message