KassieZ commented on code in PR #1977: URL: https://github.com/apache/doris-website/pull/1977#discussion_r1944304364
########## docs/sql-manual/sql-statements/cluster-management/instance-management/ADD-BROKER.md: ########## @@ -27,34 +27,43 @@ under the License. - - ## Description -This statement is used to add a BROKER node. (Administrator only!) +This statement is used to add one or more BROKER nodes. -grammar: +## Syntax ```sql -ALTER SYSTEM ADD BROKER broker_name "broker_host1:broker_ipc_port1","broker_host2:broker_ipc_port2",...; +ALTER SYSTEM ADD BROKER <broker_name> "<host>:<ipc_port>" [, "host>:<ipc_port>" [, ... ] ]; ``` -## Example +## Required Parameters + +**1. `<broker_name>`** + +The name given to the added broker process. It is recommended to keep the broker_name consistent within the same cluster. + +**2. `<host>`** -1. Add two brokers +The IP of the node where the broker process needs to be added. If FQDN is enabled, use the FQDN of the node. - ```sql - ALTER SYSTEM ADD BROKER "host1:port", "host2:port"; - ``` -2. When fe enable fqdn([fqdn](../../../admin-manual/cluster-management/fqdn.md)),add one Broker +**3. `<ipc_port>`** - ```sql - ALTER SYSTEM ADD BROKER "broker_fqdn1:port"; - ``` +The PORT of the node where the broker process needs to be added, and the default value of this port is 8000. -## Keywords - ALTER, SYSTEM, ADD, FOLLOWER, ALTER SYSTEM +## Access Control Requirements +The user who executes this operation needs to have the NODE_PRIV permission. -## Best Practice +## Examples +1. Add two Brokers + +```sql +ALTER SYSTEM ADD BROKER "host1:port", "host2:port"; +``` +2. Add a Broker using FQDN + +```sql Review Comment: 需要缩进，下面序号下也是一样，都需要缩进 ########## docs/sql-manual/sql-statements/cluster-management/instance-management/ADD-BROKER.md: ########## @@ -27,34 +27,43 @@ under the License. - - ## Description -This statement is used to add a BROKER node. (Administrator only!) +This statement is used to add one or more BROKER nodes. -grammar: +## Syntax ```sql -ALTER SYSTEM ADD BROKER broker_name "broker_host1:broker_ipc_port1","broker_host2:broker_ipc_port2",...; +ALTER SYSTEM ADD BROKER <broker_name> "<host>:<ipc_port>" [, "host>:<ipc_port>" [, ... ] ]; ``` -## Example +## Required Parameters + +**1. `<broker_name>`** + +The name given to the added broker process. It is recommended to keep the broker_name consistent within the same cluster. + +**2. `<host>`** -1. Add two brokers +The IP of the node where the broker process needs to be added. If FQDN is enabled, use the FQDN of the node. - ```sql - ALTER SYSTEM ADD BROKER "host1:port", "host2:port"; - ``` -2. When fe enable fqdn([fqdn](../../../admin-manual/cluster-management/fqdn.md)),add one Broker +**3. `<ipc_port>`** - ```sql - ALTER SYSTEM ADD BROKER "broker_fqdn1:port"; - ``` +The PORT of the node where the broker process needs to be added, and the default value of this port is 8000. -## Keywords - ALTER, SYSTEM, ADD, FOLLOWER, ALTER SYSTEM +## Access Control Requirements +The user who executes this operation needs to have the NODE_PRIV permission. -## Best Practice +## Examples +1. Add two Brokers + +```sql Review Comment: 需要缩进 -- 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: commits-unsubscr...@doris.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org For additional commands, e-mail: commits-h...@doris.apache.org
