Todd Lipcon has posted comments on this change. Change subject: WIP release notes for 0.8.0 ......................................................................
Patch Set 1: (9 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 before any clients are upgraded 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 yet), rather than branches? Otherwise the content of this link won't be correct after 0.8.0 is released if there is ever an 0.8.1 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 think better to say "running versions prior to 0.8.0" Line 72: particular, scans that specify column predicates will fail. maybe offer a bit of advice: Users should upgrade Kudu servers before upgrading clients. 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" 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? 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 - better to say "tablet replica" here Line 104: to see invalid data. I dont think this "result" is correct. It resulted in the possibility that flushed data would include out-of-order transactions, triggering a crash on the next cmopaction. 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 need to be aware) -- 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: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
