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 <mi...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>
Gerrit-HasComments: Yes

Reply via email to