Yingchun Lai has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/19512 )

Change subject: [docs] Add release notes for 1.17.0
......................................................................


Patch Set 4:

(31 comments)

http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc
File docs/release_notes.adoc:

http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@55
PS3, Line 55: [[rn_1.17.0_obsoletions]]
> Should this go under the "Optimizations and Improvements" section?
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@56
PS3, Line 56:
            :
> nit: Change to "5 minutes. It's no longer necessary to keep such a long his
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@76
PS3, Line 76: [KUDU-2671]).
            :
> nit: Change to "tables. It's now possible to add ranges with their own uniq
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@87
PS3, Line 87: e.org/jira/b
> nit: Break up sentence, change to "counter. The"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@87
PS3, Line 87: .
> nit: add comma after "tablet"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@105
PS3, Line 105: s being esta
> nit: remove "meaning that"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@110
PS3, Line 110:
> If dealing with the C++ client, KUDU-3393 is more accurate as KUDU-2670 dea
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@112
PS3, Line 112: [KUDU-3351],
> nit: switch order to "C++ and Java", corresponds to Jira links.
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@116
PS3, Line 116: nk:https://issues.apache.org/jira/browse/KUDU-3375[KUD
> Kudu now exposes metrics in Prometheus format
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@139
PS3, Line 139:
             :
> for all masters and tablet servers in a Kudu cluster at once
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@236
PS3, Line 236:
> nit: change to "where"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@237
PS3, Line 237:
> nit: remove comma
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@238
PS3, Line 238:
> Is the tablet only GCed if it was created before Kudu 1.10? Maybe put a per
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@239
PS3, Line 239:
> What does this mean?
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@242
PS3, Line 242:  b
> nit: change to "where the"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@244
PS3, Line 244: a/brow
> nit: change to "server that is"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@244
PS3, Line 244: .a
> nit: change to "in the"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@247
PS3, Line 247:
> nit: change to "in the C++ client where"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@249
PS3, Line 249: a file (see li
> nit: change to "in the C++ client where"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@251
PS3, Line 251: in the C++ cli
> nit: change to "in the Java client where"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@253
PS3, Line 253: in the C++ clien
> nit: change to "in the Java client where the"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@256
PS3, Line 256:
> nit: change to "in the Java client where it"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@258
PS3, Line 258: eque
> nit: change to "in the Kudu HMS client where"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@261
PS3, Line 261:
> nit: change to "where"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@263
PS3, Line 263: ss
> nit: change to "where"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@266
PS3, Line 266:
> nit: change to "where"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@270
PS3, Line 270: wh
> nit: change to "where the"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@270
PS3, Line 270: nary
> nit: change to "flag is"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@270
PS3, Line 270:  tabl
> nit: change to "crashes"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@302
PS3, Line 302: rn_1.17.0_client_
> nit: change to "vice-versa. If not,"
Done


http://gerrit.cloudera.org:8080/#/c/19512/3/docs/release_notes.adoc@308
PS3, Line 308:
> nit: change to "library. If not,"
Done



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

Gerrit-Project: kudu
Gerrit-Branch: branch-1.17.x
Gerrit-MessageType: comment
Gerrit-Change-Id: I4381e7824dcf7f500165aa3af9dca09a86ac1dd9
Gerrit-Change-Number: 19512
Gerrit-PatchSet: 4
Gerrit-Owner: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Hongjiang Zhang <[email protected]>
Gerrit-Reviewer: Joe McDonnell <[email protected]>
Gerrit-Reviewer: KeDeng <[email protected]>
Gerrit-Reviewer: Khazar Mammadli <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Kurt Deschler <[email protected]>
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Riza Suminto <[email protected]>
Gerrit-Reviewer: Wang Xixu <[email protected]>
Gerrit-Reviewer: Wenzhe Zhou <[email protected]>
Gerrit-Reviewer: Yifan Zhang <[email protected]>
Gerrit-Reviewer: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Yuqi Du <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>
Gerrit-Reviewer: bsglz <[email protected]>
Gerrit-Reviewer: harinreddy <[email protected]>
Gerrit-Reviewer: yejiabao <[email protected]>
Gerrit-Reviewer: zhanghongbin <[email protected]>
Gerrit-Reviewer: Ádám Bakai <[email protected]>
Gerrit-Comment-Date: Sun, 05 Mar 2023 15:07:49 +0000
Gerrit-HasComments: Yes

Reply via email to