Mike Percy has posted comments on this change. Change subject: KUDU-1517 Implement doc feedback from Sue M ......................................................................
Patch Set 1: (11 comments) http://gerrit.cloudera.org:8080/#/c/3638/1/docs/index.adoc File docs/index.adoc: PS1, Line 73: mechanism s/mechanisms/syntax/ PS1, Line 74: any other table with HDFS or HBase persistenc as any other Impala table, like those using HDFS or HBase for persistence. PS1, Line 78: solutions s/solutions/standards/ PS1, Line 78: to s/to/with/ PS1, Line 79: FROM I would say: you can specify complex joins with a FROM clause in a subquery However, I am not sure if it's true that this works for updates with Impala and Kudu. Are you sure? I suppose this means you can do something like what is shown in this SO article: http://stackoverflow.com/questions/1293330/how-can-i-do-an-update-statement-with-join-in-sql PS1, Line 89: within s/within/used by/ PS1, Line 89: to s/to/across/ Line 108: Data Compression:: Because a given column contains only one type of data, pattern-based compression can nit: Would be nice to keep this line under 100 chars long PS1, Line 185: when using s/when using/in/ PS1, Line 236: and nit: and and http://gerrit.cloudera.org:8080/#/c/3638/1/docs/release_notes.adoc File docs/release_notes.adoc: Line 31: == Introduction nit: I think this should be a 3rd-level heading -- To view, visit http://gerrit.cloudera.org:8080/3638 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I8a7647b3e5d4d36e82e06ce02a45a8811e4efed3 Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Misty Stanley-Jones <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-HasComments: Yes
