Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15685 )
Change subject: WIP [docs] Kudu 1.12 release notes draft ...................................................................... Patch Set 5: (11 comments) http://gerrit.cloudera.org:8080/#/c/15685/5/docs/release_notes.adoc File docs/release_notes.adoc: http://gerrit.cloudera.org:8080/#/c/15685/5/docs/release_notes.adoc@56 PS5, Line 56: == New features > I am not sure we want to add a note about columnar given it's experimental Do you think it's not worth documenting the auto rebalancer, given it's also marked experimental and defaulted to off? http://gerrit.cloudera.org:8080/#/c/15685/5/docs/release_notes.adoc@75 PS5, Line 75: If setting : `--time_source=auto`, Kudu masters and tablet servers use the built-in NTP : client synchronized with dedicated NTP servers available via host-only : network in AWS and GCE cloud environments, otherwise Kudu masters and tablet : servers rely on local machine's clock synchronized by NTP > I think this sentence needs to be broken up. E.g.: Done http://gerrit.cloudera.org:8080/#/c/15685/5/docs/release_notes.adoc@85 PS5, Line 85: catalog manager > I think changing this to "masters" would make it easier to understand for u Done http://gerrit.cloudera.org:8080/#/c/15685/5/docs/release_notes.adoc@93 PS5, Line 93: [[rn_1.12.0_improvements]] > 384a535a0 [cfile] KUDU-2852 Push predicate evaluation for int type RLE deco Done http://gerrit.cloudera.org:8080/#/c/15685/5/docs/release_notes.adoc@105 PS5, Line 105: * Kudu tablet servers and masters now expose a metric for the number of tablet > should we mention the name of the metric here? Done http://gerrit.cloudera.org:8080/#/c/15685/5/docs/release_notes.adoc@118 PS5, Line 118: * The built-in NTP client is now fully supported as the time source for Kudu's > I think this section and section about chronyd should be moved up right abo I'm on the fence about this. I'll leave it as is -- others that may feel more strongly about this, feel free to update. http://gerrit.cloudera.org:8080/#/c/15685/5/docs/release_notes.adoc@147 PS5, Line 147: [[rn_1.12.0_fixed_issues]] > 34efee128 [master] KUDU-2904 Crash master on disk error Done http://gerrit.cloudera.org:8080/#/c/15685/5/docs/release_notes.adoc@217 PS5, Line 217: * Andy Singer > 90c12dc27 [java] KUDU-3035: Pass last propagated timestamp in Batch Done http://gerrit.cloudera.org:8080/#/c/15685/5/docs/release_notes.adoc@219 PS5, Line 219: * Ning Wang > 1b3b26d95 KUDU-3070 skip open block manager Done http://gerrit.cloudera.org:8080/#/c/15685/5/docs/release_notes.adoc@222 PS5, Line 222: * Thomas D'Silva > b58b37a1f KUDU-2162 Expose stats about scan filters Done http://gerrit.cloudera.org:8080/#/c/15685/5/docs/release_notes.adoc@224 PS5, Line 224: * Waleed Fateem > 5432d316a KUDU-3099: Remove System.exit() calls from KuduBackup/KuduRestore Done -- To view, visit http://gerrit.cloudera.org:8080/15685 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.12.x Gerrit-MessageType: comment Gerrit-Change-Id: I300fb597a4eed36199ebf8760084e6df1fb04e9a Gerrit-Change-Number: 15685 Gerrit-PatchSet: 5 Gerrit-Owner: Hao Hao <[email protected]> Gerrit-Reviewer: Adar Lieber-Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Attila Bukor <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Greg Solovyev <[email protected]> Gerrit-Reviewer: Hannah Nguyen <[email protected]> Gerrit-Reviewer: Hao Hao <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Volodymyr Verovkin <[email protected]> Gerrit-Comment-Date: Thu, 16 Apr 2020 23:29:00 +0000 Gerrit-HasComments: Yes
