Attila Bukor has posted comments on this change. ( http://gerrit.cloudera.org:8080/11647 )
Change subject: Add release notes for 1.8.0 ...................................................................... Patch Set 4: (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. Done http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc@52 PS2, Line 52: - The `-table_num_buckets` configuration option of the `kudu perf loadgen` tool is now > This should clarify that it applies to the loadgen tool, otherwise it can b I think it's still a good idea to list it in deprecations too, reworded it to make sure it's clear it's only for `kudu perf loadgen` http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc@66 PS2, Line 66: > perhaps this should link to github? Done http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc@94 PS2, Line 94: > I'd add 'in a scan' to clarify the context Done http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc@144 PS2, Line 144: > This is a better description of the change I pointed out above, perhaps tha Done http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc@173 PS2, Line 173: > pretty sure this sentence is a fragment, it should include a description of Done http://gerrit.cloudera.org:8080/#/c/11647/2/docs/release_notes.adoc@235 PS2, Line 235: - Kudu 1.8 clients may connect to servers running Kudu 1.0 or later. If the client uses > This is largely repeating what was said above, should it be consolidated? I removed this and left the one at the top -- 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: 4 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 18:15:30 +0000 Gerrit-HasComments: Yes
