zhongjiajie commented on a change in pull request #6681:
URL: https://github.com/apache/dolphinscheduler/pull/6681#discussion_r742577269
##########
File path: dolphinscheduler-server/src/main/resources/config/install_config.conf
##########
@@ -30,29 +30,29 @@ sshPort="22"
# A comma separated list of machine hostname or IP would be installed Master
server, it
# must be a subset of configuration `ips`.
-# Example for hostnames: ips="ds1,ds2", Example for IP:
ips="192.168.8.1,192.168.8.2"
+# Example for hostname: masters="ds1,ds2", Example for IP:
masters="192.168.8.1,192.168.8.2"
masters="ds1,ds2"
# A comma separated list of machine <hostname>:<workerGroup> or
<IP>:<workerGroup>.All hostname or IP must be a
# subset of configuration `ips`, And workerGroup have default value as
`default`, but we recommend you declare behind the hosts
-# Example for hostnames: ips="ds1:default,ds2:default,ds3:default", Example
for IP: ips="192.168.8.1:default,192.168.8.2:default,192.168.8.3:default"
+# Example for hostname: workers="ds1:default,ds2:default,ds3:default", Example
for IP: workers="192.168.8.1:default,192.168.8.2:default,192.168.8.3:default"
workers="ds1:default,ds2:default,ds3:default,ds4:default,ds5:default"
# A comma separated list of machine hostname or IP would be installed Alert
server, it
# must be a subset of configuration `ips`.
-# Example for hostnames: ips="ds3", Example for IP: ips="192.168.8.3"
+# Example for hostname: alertServer="ds3", Example for IP:
alertServer="192.168.8.3"
alertServer="ds3"
# A comma separated list of machine hostname or IP would be installed API
server, it
# must be a subset of configuration `ips`.
-# Example for hostnames: ips="ds1", Example for IP: ips="192.168.8.1"
+# Example for hostname: apiServers="ds1", Example for IP:
apiServers="192.168.8.1"
apiServers="ds1"
-# The directory to install DolphinScheduler for all machine we config above.
It will automatically created by `install.sh` script if not exists.
-# **DO NOT** set this configuration same as the current path (pwd)
+# The directory to install DolphinScheduler for all machine we config above.
It will automatically be created by `install.sh` script if not exists.
+# Do not set this configuration same as the current path (pwd)
Review comment:
`**DO NOT**` it just like highlight as as html, I think is better to
keep it
##########
File path: dolphinscheduler-server/src/main/resources/config/install_config.conf
##########
@@ -21,7 +21,7 @@
# A comma separated list of machine hostname or IP would be installed
DolphinScheduler,
# including master, worker, api, alert. If you want to deploy in
pseudo-distributed
# mode, just write a pseudo-distributed hostname
-# Example for hostnames: ips="ds1,ds2,ds3,ds4,ds5", Example for IP:
ips="192.168.8.1,192.168.8.2,192.168.8.3,192.168.8.4,192.168.8.5"
+# Example for hostname: ips="ds1,ds2,ds3,ds4,ds5", Example for IP:
ips="192.168.8.1,192.168.8.2,192.168.8.3,192.168.8.4,192.168.8.5"
Review comment:
I thinks using plural here is good, cause we have multiple hostname
assinged to it. Same as below changed you make, WDYT
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
