Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/16410 )
Change subject: Add release notes for 1.13.0 ...................................................................... Patch Set 1: (10 comments) http://gerrit.cloudera.org:8080/#/c/16410/1/docs/release_notes.adoc File docs/release_notes.adoc: http://gerrit.cloudera.org:8080/#/c/16410/1/docs/release_notes.adoc@49 PS1, Line 49: Tables have owners now Maybe "Added table ownership support" http://gerrit.cloudera.org:8080/#/c/16410/1/docs/release_notes.adoc@50 PS1, Line 50: and ... by altering the table. You can also ... http://gerrit.cloudera.org:8080/#/c/16410/1/docs/release_notes.adoc@64 PS1, Line 64: Map Reduce Remove since it's deprecated and we likely wont ever add support. http://gerrit.cloudera.org:8080/#/c/16410/1/docs/release_notes.adoc@88 PS1, Line 88: A new metric introduced for both Masters and Tablet Servers: number of RPC : requests of a particular type dropped due to RPC service queue overflow. Maybe specify the exact metric? http://gerrit.cloudera.org:8080/#/c/16410/1/docs/release_notes.adoc@90 PS1, Line 90: A CoDel-like queue control mechanism for the apply queue is introduced. Introduced a CoDel-like queue control mechanism for the apply queue. http://gerrit.cloudera.org:8080/#/c/16410/1/docs/release_notes.adoc@92 PS1, Line 92: Many occurrences of 2 spaces after periods. I think we usually use 1. Either way, we should be consistent. http://gerrit.cloudera.org:8080/#/c/16410/1/docs/release_notes.adoc@97 PS1, Line 97: Kudu servers can now use Unix domain sockets for RPC communication. As of : now, this is mostly used in tests but in future it might be used for : “zero-copy” communication between Kudu servers and clients collocated at the : same node. Is this worth release noting if it's test only right now? http://gerrit.cloudera.org:8080/#/c/16410/1/docs/release_notes.adoc@130 PS1, Line 130: increases was increased http://gerrit.cloudera.org:8080/#/c/16410/1/docs/release_notes.adoc@147 PS1, Line 147: - extra hyphen? http://gerrit.cloudera.org:8080/#/c/16410/1/docs/release_notes.adoc@150 PS1, Line 150: that to -- To view, visit http://gerrit.cloudera.org:8080/16410 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.13.x Gerrit-MessageType: comment Gerrit-Change-Id: Iac2a0ae740c3dabc5e7d8b7ef53312924c6e3532 Gerrit-Change-Number: 16410 Gerrit-PatchSet: 1 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Adar Lieber-Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Anonymous Coward (314) Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Greg Solovyev <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Thu, 03 Sep 2020 16:53:39 +0000 Gerrit-HasComments: Yes
