Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15456 )
Change subject: WIP [release notes] supported NTP servers and new time source ...................................................................... Patch Set 1: (4 comments) http://gerrit.cloudera.org:8080/#/c/15456/1//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/15456/1//COMMIT_MSG@13 PS1, Line 13: drop the whole : mention about the newly introduced time source completely? +1. Let's not tell people about it at all. http://gerrit.cloudera.org:8080/#/c/15456/1/docs/release_notes.adoc File docs/release_notes.adoc: http://gerrit.cloudera.org:8080/#/c/15456/1/docs/release_notes.adoc@48 PS1, Line 48: == Optimizations and improvements I might also doc that the built-in client is now fully supported (i.e. not experimental), and that --time_source should be used to switch between it and the existing system source. http://gerrit.cloudera.org:8080/#/c/15456/1/docs/release_notes.adoc@49 PS1, Line 49: `chronyd` of version 3.4 and newer Nit: `chronyd` (version 3.4 and newer) is now supported... http://gerrit.cloudera.org:8080/#/c/15456/1/docs/release_notes.adoc@50 PS1, Line 50: In other words, : with contemporary Linux distributions it's no longer necessary to replace : `chronyd` with `ntpd` when running Kudu. I don't think we need to say this. -- To view, visit http://gerrit.cloudera.org:8080/15456 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I70c2075c73693cc9605372ea5b58044ebbf93bdf Gerrit-Change-Number: 15456 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Tue, 17 Mar 2020 05:08:08 +0000 Gerrit-HasComments: Yes
