impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matthew Jacobs (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-4403: Implement SHOW RANGE PARTITIONS for Kudu tables
Date Wed, 07 Dec 2016 19:46:55 GMT
Matthew Jacobs has posted comments on this change.

Change subject: IMPALA-4403: Implement SHOW RANGE PARTITIONS for Kudu tables

Patch Set 4:

Commit Message:

PS4, Line 9: This is a rough draft and should serve as a base for discussion on how
           : to properly implement this. I'm happy to add another command to the
           : catalog to handle show partitions separately from show stats. We might
           : as well merge this change now and then clean it up properly in a
           : subsequent change.
           : Also I'm happy to add more tests if needed, e.g. to the AnalyzerTests.
can you update this now?
File testdata/workloads/functional-query/queries/QueryTest/kudu_partition_ddl.test:

PS4, Line 97: 'VALUES = (10, "martin")'
            : 'VALUES = (20, "dimitris")'
            : 'VALUES = (30, "matthew")'
did you fetch the latest kudu client jar from mvn? VALUES should be VALUE. We may need to
wait to get this in so we don't break tests when the mvn artifact changes
File tests/query_test/

PS4, Line 133:         assert False

To view, visit
To unsubscribe, visit

Gerrit-MessageType: comment
Gerrit-Change-Id: Idf5b2fdd02938a42fa59ec98884e4ac915dd1f65
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Lars Volker <>
Gerrit-Reviewer: Alex Behm <>
Gerrit-Reviewer: Dan Burkert <>
Gerrit-Reviewer: Lars Volker <>
Gerrit-Reviewer: Matthew Jacobs <>
Gerrit-HasComments: Yes

View raw message