Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/9685 )
Change subject: [docs] Add remaining release notes for 1.7.0 ...................................................................... Patch Set 1: (18 comments) http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc File docs/release_notes.adoc: http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@112 PS1, Line 112: and enabled by default , which is enabled by default http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@117 PS1, Line 117: see Extra word http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@148 PS1, Line 148: If disabled, Spark task will be only scheduled on : leader replica. Does this mean tasks that scan a Kudu tablet will be scheduled only on the leader replica? I think Spark can still schedule the task anywhere, it just is more likely to schedule it local to the data. Maybe for this release note we should say something like "By default, Spark tasks that scan Kudu will now be able to scan non-leader replicas. This allows Spark to more easily schedule kudu-spark tasks local to the data. Users can disable this behavior by passing 'leader_only' to the 'kudu.scanLocaliity' option." http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@170 PS1, Line 170: Sparse column predicate can cause excessive data-block reads This sounds like it's doc'ing a known issue. Maybe we want to begin "An issue where sparse column predicates could cause excessive data-block reads has been fixed"? http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@197 PS1, Line 197: CA certificate and authentication token a CA certificate and an authentication token http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@197 PS1, Line 197: Leader master The leader master now http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@198 PS1, Line 198: a http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@198 PS1, Line 198: Kudu clients a Kudu client http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@199 PS1, Line 199: a Extra "a" http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@200 PS1, Line 200: a http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@209 PS1, Line 209: * Kudu Java client now will retry a connection if no master is discovered as a The http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@210 PS1, Line 210: a http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@212 PS1, Line 212: started starting http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@224 PS1, Line 224: has had http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@223 PS1, Line 223: been : leaders since the very start Meaning, if they haven't ever been leader? http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@231 PS1, Line 231: a http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@231 PS1, Line 231: a http://gerrit.cloudera.org:8080/#/c/9685/1/docs/release_notes.adoc@241 PS1, Line 241: Allow move tool to work when uninvolved tserver is down. I think this sentence can be removed. -- To view, visit http://gerrit.cloudera.org:8080/9685 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I1718a2065663245a5bc438288013871566d6941b Gerrit-Change-Number: 9685 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Fri, 16 Mar 2018 04:34:41 +0000 Gerrit-HasComments: Yes
