Misty Stanley-Jones has posted comments on this change. Change subject: Add docs for non-covering range partitioning ......................................................................
Patch Set 1: (6 comments) http://gerrit.cloudera.org:8080/#/c/3796/1/docs/kudu_impala_integration.adoc File docs/kudu_impala_integration.adoc: Line 830: range should be added. APIs `AlterTableRequestPB::ADD_RANGE_PARTITION` and > This keyword is the specific Protobuf identifier that Kudu uses internally; Done http://gerrit.cloudera.org:8080/#/c/3796/1/docs/schema_design.adoc File docs/schema_design.adoc: Line 49: across tablet servers. This is most impacted by the primary key design and the > This is only influenced by the partition schema; the primary key only influ Done Line 53: - Only important data would be transmitted to clients, by means of encoding, compression, > I think what this is trying to get at is that an efficient schema will be d Done Line 147: Kudu 0.10 introduces non-covering range partitions, to mitigate some limitations of > This doesn't explain what non-covering range partitions are. Done Line 165: ==== Caveats of Non-Covering Range Partitions > These caveats look like they were taken right from the design doc? I think Done Line 288: [[column-design]] > Are there changes in here, or just a straight move? Just a move. -- To view, visit http://gerrit.cloudera.org:8080/3796 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I3b0fd7500c5399db9dcad617ae67fea247307353 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Misty Stanley-Jones <mi...@apache.org> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Dinesh Bhat <din...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Misty Stanley-Jones <mi...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-HasComments: Yes