Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15456 )

Change subject: WIP [release notes] supported NTP servers and time source
......................................................................


Patch Set 2:

(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:
            :
> +1. Let's not tell people about it at all.
Thank you for the feedback!


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
Done


http://gerrit.cloudera.org:8080/#/c/15456/1/docs/release_notes.adoc@49
PS1, Line 49: The built-in NTP client is now ful
> Nit: `chronyd` (version 3.4 and newer) is now supported...
Done


http://gerrit.cloudera.org:8080/#/c/15456/1/docs/release_notes.adoc@50
PS1, Line 50: l. To switch the
            :   time source from the existing `system` time source (which is 
the default)
            :   to the built-in NTP client, use `--time_
> I don't think we need to say this.
Done



-- 
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: 2
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 07:55:48 +0000
Gerrit-HasComments: Yes

Reply via email to