Adar Dembo has posted comments on this change. ( http://gerrit.cloudera.org:8080/15238 )
Change subject: [mini-cluster] built-in NTP client configuration mode ...................................................................... Patch Set 1: (3 comments) http://gerrit.cloudera.org:8080/#/c/15238/1/src/kudu/mini-cluster/external_mini_cluster.h File src/kudu/mini-cluster/external_mini_cluster.h: http://gerrit.cloudera.org:8080/#/c/15238/1/src/kudu/mini-cluster/external_mini_cluster.h@101 PS1, Line 101: to configure : // the Replace with "for its" http://gerrit.cloudera.org:8080/#/c/15238/1/src/kudu/mini-cluster/external_mini_cluster.h@112 PS1, Line 112: robing robin http://gerrit.cloudera.org:8080/#/c/15238/1/src/kudu/mini-cluster/external_mini_cluster.h@233 PS1, Line 233: // The way how the built-in NTP client is configured given the set of NTP : // servers run by the mini-cluster. Replace with "Mapping of NTP servers to built-in NTP client for each Kudu server". Also note that it's only relevant if num_ntp_servers > 0. -- To view, visit http://gerrit.cloudera.org:8080/15238 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I88561f757678b664b810b54a27317cac99297403 Gerrit-Change-Number: 15238 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Volodymyr Verovkin <[email protected]> Gerrit-Comment-Date: Wed, 19 Feb 2020 02:07:44 +0000 Gerrit-HasComments: Yes
