Todd Lipcon has posted comments on this change. ( http://gerrit.cloudera.org:8080/13662 )
Change subject: [docs] Add 1.10.0 release notes ...................................................................... Patch Set 1: (9 comments) http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc File docs/release_notes.adoc: http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@49 PS1, Line 49: Spark backup I think this phrasing is a little confusing. Perhaps a backup job implemented using Apache Spark? Or "a job which can run on an Apache Spark cluster"? or "using Apache Spark as an execution environment" or something? As it's written it sounds like a job which backs up Spark http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@55 PS1, Line 55: HMS synchronization documentation : for more details. can this be a link? http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@61 PS1, Line 61: authorization documentation link? Worth noting that this only works when HMS integration is enabled (right? or can it be enabled independently?) http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@69 PS1, Line 69: though this : may be overridden using `--max_column_comment_length` Do we want to document this? We didn't tag this flag as stable (though I guess we also didn't call it unsafe or experimental). http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@83 PS1, Line 83: helm Helm? http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@83 PS1, Line 83: statefulset I think this is camel-cased StatefulSet http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@96 PS1, Line 96: * `kudu table copy` copies data from one table to another, within the same : cluster or across clusters. Do we want to say something here to indicate this is driven by a single client and therefore not suitable for large tables? http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@113 PS1, Line 113: * The performance of mutations (i.e. UPDATE, DELETE, and re-INSERT) to can this be combined with the "Update performance" one above? http://gerrit.cloudera.org:8080/#/c/13662/1/docs/release_notes.adoc@137 PS1, Line 137: Once resolved DNS records I think it's simpler phrasing to write "DNS records are now cached" or "DNS resolutions are now cached" -- To view, visit http://gerrit.cloudera.org:8080/13662 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I040df9debdf78f1bd932915164a1b6ba490be135 Gerrit-Change-Number: 13662 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: Grant Henke <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Mon, 17 Jun 2019 18:16:56 +0000 Gerrit-HasComments: Yes
