Todd Lipcon has posted comments on this change. Change subject: Doc review for 1.2 ......................................................................
Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/5676/1/docs/kudu_impala_integration.adoc File docs/kudu_impala_integration.adoc: PS1, Line 448: DISTRIBUTE BY RANGE (_id) > I believe that's a misconception double checked, I can do: [vd0342.halxg.cloudera.com:21000] > create table todd_test (a int primary key) partition by range(a) ( partition values < 100, partition 100 <= values) stored as kudu; PS1, Line 482: '\<', '\>', `>=`, or `IN` > oh, yea, BETWEEN should get pushed Done PS1, Line 941: 8 > ah, didn't know that. Done -- To view, visit http://gerrit.cloudera.org:8080/5676 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I64ad4925ba4402ef3733a0ab62da91f1049848fb Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Ambreen Kazi <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: John Russell <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
