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

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


Patch Set 8:

(10 comments)

> It has been a very long time since we start the Kudu 1.17 release,
 > I'm very sorry for the late update. Thank you all to review this
 > patch again.

Ah, that's not a problem.

Thank you very much for working on RM for this release.  I really appreciate 
this!

I added some comments for PS6, but probably they are not up-to-date any longer. 
 If you find them relevant, please update the text.  Otherwise, I'm going to 
take a fresh new look at PS8.

Thanks a lot!

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

http://gerrit.cloudera.org:8080/#/c/19512/6/docs/release_notes.adoc@73
PS6, Line 73: table wide
nit: table-wide


http://gerrit.cloudera.org:8080/#/c/19512/6/docs/release_notes.adoc@78
PS6, Line 78: soft-delete tables
nit: 'soft deletion of tables' or 'soft-deleted tables'


http://gerrit.cloudera.org:8080/#/c/19512/6/docs/release_notes.adoc@80
PS6, Line 80: delete table
DeleteTable


http://gerrit.cloudera.org:8080/#/c/19512/6/docs/release_notes.adoc@90
PS6, Line 90: will be
is


http://gerrit.cloudera.org:8080/#/c/19512/6/docs/release_notes.adoc@103
PS6, Line 103: Introduced JWT verification into C++ RPC negotiation
Introduced an experimental feature: authentication using JWT tokens ...


http://gerrit.cloudera.org:8080/#/c/19512/6/docs/release_notes.adoc@104
PS6, Line 104: `--enable_jwt_token_auth` has been enabled
`--enable_jwt_token_auth` is set to `true`


http://gerrit.cloudera.org:8080/#/c/19512/6/docs/release_notes.adoc@108
PS6, Line 108: call `SetSplitSizeBytes()`
use the newly introduced `SetSplitSizeBytes()` method of the Kudu client API


http://gerrit.cloudera.org:8080/#/c/19512/6/docs/release_notes.adoc@115
PS6, Line 115: via `/metrics_prometheus` path of the webserver
via the embedded webserver's `/metrics_prometheus` endpoint


http://gerrit.cloudera.org:8080/#/c/19512/6/docs/release_notes.adoc@115
PS6, Line 115: metrics
all its metrics except for string gauges


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

http://gerrit.cloudera.org:8080/#/c/19512/8/docs/release_notes.adoc@240
PS8, Line 240:
https://issues.apache.org/jira/browse/KUDU-2906



--
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: 8
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: Tue, 13 Jun 2023 05:45:37 +0000
Gerrit-HasComments: Yes

Reply via email to