Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/13669 )

Change subject: [docs] Add 1.10.0 release notes
......................................................................


Patch Set 2:

(10 comments)

http://gerrit.cloudera.org:8080/#/c/13669/1/docs/release_notes.adoc
File docs/release_notes.adoc:

http://gerrit.cloudera.org:8080/#/c/13669/1/docs/release_notes.adoc@70
PS1, Line 70:
> nit: extra period
Done


http://gerrit.cloudera.org:8080/#/c/13669/1/docs/release_notes.adoc@97
PS1, Line 97: erages a sing
> nit: "single-client"? "leverages a single client"?
Done


http://gerrit.cloudera.org:8080/#/c/13669/1/docs/release_notes.adoc@123
PS1, Line 123: `tableExists()` (Java) and `TableExist
> nit: surround code in backticks
Done


http://gerrit.cloudera.org:8080/#/c/13669/1/docs/release_notes.adoc@130
PS1, Line 130: speed-up
> nit: speed-up
Done


http://gerrit.cloudera.org:8080/#/c/13669/1/docs/release_notes.adoc@169
PS1, Line 169: `PartialRow` and `RowRes
> nit: surround code in backticks
Done


http://gerrit.cloudera.org:8080/#/c/13669/1/docs/release_notes.adoc@183
PS1, Line 183: `/metric
> Should this have backticks?
Done


http://gerrit.cloudera.org:8080/#/c/13669/1/docs/release_notes.adoc@186
PS1, Line 186: on a tablet server.
             :
> nit: on a tablet server?
Done


http://gerrit.cloudera.org:8080/#/c/13669/1/docs/release_notes.adoc@201
PS1, Line 201: A value of 0
             :   now indicates that no rows should be generated, and a value of 
-1 indicates
             :   there should be no limit to the number of rows generated.
             :
> How about:
Done


http://gerrit.cloudera.org:8080/#/c/13669/1/docs/release_notes.adoc@206
PS1, Line 206: aries into app
> nit: surround code in backticks
Done


http://gerrit.cloudera.org:8080/#/c/13669/1/docs/release_notes.adoc@298
PS1, Line 298: * Florentino Sainz
             : * Guangchao Deng
             : * Jia Hongchao
             : * Ye Yuqiang
             : * Yifan Zhang
             :
             : Thank you for your help in making Kudu even better!
             :
             : [[resources_and_next_steps]]
             : == Resources
             :
> Should this be a bulleted list?
It wasn't in past releases, but that probably looks better.



--
To view, visit http://gerrit.cloudera.org:8080/13669
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.10.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I040df9debdf78f1bd932915164a1b6ba490be135
Gerrit-Change-Number: 13669
Gerrit-PatchSet: 2
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Thu, 20 Jun 2019 15:40:01 +0000
Gerrit-HasComments: Yes

Reply via email to