Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/18275 )
Change subject: [docs] Add release notes for 1.16.0 ...................................................................... Patch Set 8: (5 comments) http://gerrit.cloudera.org:8080/#/c/18275/6/docs/release_notes.adoc File docs/release_notes.adoc: http://gerrit.cloudera.org:8080/#/c/18275/6/docs/release_notes.adoc@92 PS6, Line 92: SELECT COUNT(1) > nit: surround in back-ticks. Same elsewhere for consistency Done http://gerrit.cloudera.org:8080/#/c/18275/6/docs/release_notes.adoc@117 PS6, Line 117: > Is this referring to this patch[1]? It shouldn't be specific to cmeta files Yep, thanks for pointing this out, not sure why I wrote cmeta instead of PBC. http://gerrit.cloudera.org:8080/#/c/18275/6/docs/release_notes.adoc@130 PS6, Line 130: ira/browse/KUDU-3311[KUDU-3311]). : > nit: how about "... start up a master when there is an additional master ad Done http://gerrit.cloudera.org:8080/#/c/18275/7/docs/release_notes.adoc File docs/release_notes.adoc: http://gerrit.cloudera.org:8080/#/c/18275/7/docs/release_notes.adoc@40 PS7, Line 40: New features > Just a small observation after looking at the 'New features' sections from Do you mean you consider all of these changes improvements rather than new features? Should we just skip the new features section? New tools, for example, are listed as new features in 1.12.0 release notes: https://kudu.apache.org/releases/1.12.0/docs/release_notes.html http://gerrit.cloudera.org:8080/#/c/18275/7/docs/release_notes.adoc@152 PS7, Line 152: y. See > nit: replace with 'unresolvable' ? Done -- To view, visit http://gerrit.cloudera.org:8080/18275 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.16.x Gerrit-MessageType: comment Gerrit-Change-Id: I0beb2e182af261ce785cfe21c7de34ca953e6a32 Gerrit-Change-Number: 18275 Gerrit-PatchSet: 8 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Hongjiang Zhang <[email protected]> Gerrit-Reviewer: KeDeng <[email protected]> Gerrit-Reviewer: Khazar Mammadli <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Riza Suminto <[email protected]> Gerrit-Reviewer: Will Berkeley <[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: yejiabao <[email protected]> Gerrit-Comment-Date: Tue, 01 Mar 2022 23:22:54 +0000 Gerrit-HasComments: Yes
