Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/11647 )
Change subject: Add release notes for 1.8.0 ...................................................................... Patch Set 2: (7 comments) http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc File docs/release_notes.adoc: http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc@36 PS2, Line 36: Kudu, it is recommended to first shut down all Kudu processes across the cluster, then Here and elsewhere the indenting is lost. http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc@52 PS2, Line 52: - The `-table_num_buckets` configuration is now deprecated in favor of This should clarify that it applies to the loadgen tool, otherwise it can be misinterpreted to be a much larger change. edit: see comment on line 144 below http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc@66 PS2, Line 66: `examples/` perhaps this should link to github? http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc@94 PS2, Line 94: - Clients now support setting a limit on the number of returned rows (see I'd add 'in a scan' to clarify the context http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc@144 PS2, Line 144: - The `kudu perf loadgen` tool now supports generating range-partitioned tables. The This is a better description of the change I pointed out above, perhaps that one should just be removed. http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc@173 PS2, Line 173: When a tablet server was wiped and recreated with the same RPC address pretty sure this sentence is a fragment, it should include a description of what failed. http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc@235 PS2, Line 235: - Rolling upgrade between Kudu 1.7 and Kudu 1.8 servers is believed to be possible though This is largely repeating what was said above, should it be consolidated? -- To view, visit http://gerrit.cloudera.org:8080/11647 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.8.x Gerrit-MessageType: comment Gerrit-Change-Id: I15b0ce686c5e69648fe09a18ca82b9bf54cab837 Gerrit-Change-Number: 11647 Gerrit-PatchSet: 2 Gerrit-Owner: Attila Bukor <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alex Rodoni <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Attila Piros <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Ferenc Szabo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Greg Solovyev <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Mitch Barnett Gerrit-Reviewer: Sailesh Mukil <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Wed, 10 Oct 2018 17:32:09 +0000 Gerrit-HasComments: Yes
