Misty Stanley-Jones has posted comments on this change. Change subject: KUDU-1382 release notes for 0.8.0 ......................................................................
Patch Set 1: (10 comments) http://gerrit.cloudera.org:8080/#/c/2655/1/docs/installation.adoc File docs/installation.adoc: Line 644: TODO find a nice way to say what's in the release notes. > should link to it and specifically advise that the servers be upgraded befo Done http://gerrit.cloudera.org:8080/#/c/2655/1/docs/release_notes.adoc File docs/release_notes.adoc: Line 63: for Kudu 0.8.0</a>+++ and +++<a href="https://github.com/apache/incubator-kudu/compare/branch-0.7.x...branch-0.8.x";>Git > should this link compare specific release tags (even if they don't exist ye Done Line 71: . 0.8.0 clients will generally not be compatible with servers not running at least 0.8.0. In > s/will generally not be/are not fully/ I know it doesn't matter but the Cloudera style guide says to use 'or higher / or lower' with versions. In the absence of strong feeling, I'll go with that. Line 72: particular, scans that specify column predicates will fail. > maybe offer a bit of advice: Users should upgrade Kudu servers before upgra Done Line 80: ==== Improvements > Another one is KUDU-839. Done Line 83: was leveraged in order to add partition pruning for hash-bucketed tables, so that we don't try to > leveraged? how bout "done" Done Line 84: read from tablets that we know don't serve the row keys we're looking for. The latter was done in > I think this could be written more clearly. Misty? Done Line 87: . The C++ client will now retry write operations if the tablet it was trying to reach was already > - I'd say this is a bug fix Done Line 104: to see invalid data. > I dont think this "result" is correct. It resulted in the possibility that Done Line 112: . The concept of "feature flags" was introduced in order to manage compability between different > we should specify that this is an internal change (users should not really Done -- To view, visit http://gerrit.cloudera.org:8080/2655 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I4e83f314cbb373f05b044bd46602713c7dbaee3e Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Jean-Daniel Cryans Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Misty Stanley-Jones <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
