morrySnow commented on code in PR #1884: URL: https://github.com/apache/doris-website/pull/1884#discussion_r1923811507
########## docs/sql-manual/sql-statements/Cluster-Management-Statements/ALTER-SYSTEM-ADD-BROKER.md: ########## @@ -25,38 +25,44 @@ specific language governing permissions and limitations under the License. --> -## ALTER-SYSTEM-ADD-BROKER +## Description -### Name +This statement is used to add one or more BROKER nodes. (For administrators only!) -ALTER SYSTEM ADD BROKER +## Syntax -### Description +```sql +ALTER SYSTEM ADD BROKER broker_name "<host1>:<ipc_port>","<host2>:<ipc_port>", ...; Review Comment: ```suggestion ALTER SYSTEM ADD BROKER <broker_name> "<host1>:<ipc_port>","<host2>:<ipc_port>", ...; ``` ########## docs/sql-manual/sql-statements/Cluster-Management-Statements/ALTER-SYSTEM-DROP-BROKER.md: ########## @@ -25,42 +25,48 @@ specific language governing permissions and limitations under the License. --> -## ALTER-SYSTEM-DROP-BROKER +## Description -### Name +This statement is used to delete BROKER nodes. (For administrators only) -ALTER SYSTEM DROP BROKER +## Syntax -### Description - -This statement is to delete the BROKER node, (administrator only) - -grammar: +1. Drop all Brokers +```sql +ALTER SYSTEM DROP ALL BROKER broker_name; Review Comment: ```suggestion ALTER SYSTEM DROP ALL BROKER <broker_name>; ``` ########## docs/sql-manual/sql-statements/Cluster-Management-Statements/ALTER-SYSTEM-ADD-BROKER.md: ########## @@ -25,38 +25,44 @@ specific language governing permissions and limitations under the License. --> -## ALTER-SYSTEM-ADD-BROKER +## Description -### Name +This statement is used to add one or more BROKER nodes. (For administrators only!) -ALTER SYSTEM ADD BROKER +## Syntax -### Description +```sql +ALTER SYSTEM ADD BROKER broker_name "<host1>:<ipc_port>","<host2>:<ipc_port>", ...; +``` -This statement is used to add a BROKER node. (Administrator only!) +## Required Parameters -grammar: +**1. broker_name** Review Comment: ```suggestion **1. <broker_name>** ``` ########## docs/sql-manual/sql-statements/Cluster-Management-Statements/ALTER-SYSTEM-DROP-BROKER.md: ########## @@ -25,42 +25,48 @@ specific language governing permissions and limitations under the License. --> -## ALTER-SYSTEM-DROP-BROKER +## Description -### Name +This statement is used to delete BROKER nodes. (For administrators only) -ALTER SYSTEM DROP BROKER +## Syntax -### Description - -This statement is to delete the BROKER node, (administrator only) - -grammar: +1. Drop all Brokers +```sql +ALTER SYSTEM DROP ALL BROKER broker_name; +``` +2. Drop one or more Broker nodes ```sql --- Delete all brokers -ALTER SYSTEM DROP ALL BROKER broker_name --- Delete a Broker node -ALTER SYSTEM DROP BROKER broker_name "host:port"[,"host:port"...]; +ALTER SYSTEM DROP BROKER broker_name "<host1>:<ipc_port>"[,"<host2>:<ipc_port>", ...]; Review Comment: ```suggestion ALTER SYSTEM DROP BROKER <broker_name> "<broer_host>:<ipc_port>" [, "<broer_host>:<ipc_port>" [, ... ] ]; ``` ########## docs/sql-manual/sql-statements/Cluster-Management-Statements/ALTER-SYSTEM-ADD-BROKER.md: ########## @@ -25,38 +25,44 @@ specific language governing permissions and limitations under the License. --> -## ALTER-SYSTEM-ADD-BROKER +## Description -### Name +This statement is used to add one or more BROKER nodes. (For administrators only!) -ALTER SYSTEM ADD BROKER +## Syntax -### Description +```sql +ALTER SYSTEM ADD BROKER broker_name "<host1>:<ipc_port>","<host2>:<ipc_port>", ...; +``` -This statement is used to add a BROKER node. (Administrator only!) +## Required Parameters -grammar: +**1. broker_name** +The name given to the added broker process. It is recommended to keep the broker_name consistent within the same cluster. -```sql -ALTER SYSTEM ADD BROKER broker_name "broker_host1:broker_ipc_port1","broker_host2:broker_ipc_port2",...; -``` +**2. host** Review Comment: 参数列表里面，语法中有尖括号的，这里也都要加上 ########## docs/sql-manual/sql-statements/Show-Statements/SHOW-BROKER.md: ########## @@ -24,34 +24,45 @@ specific language governing permissions and limitations under the License. --> -## SHOW-BROKER +## Description -### Name +This statement is used to view the status of the currently existing broker processes. -SHOW BROKER - -### Description - -This statement is used to view the currently existing broker - -grammar: +## Syntax： ```sql SHOW BROKER; ``` -illustrate: +## Required Parameters Review Comment: 没有就不要这个章节了 ########## docs/sql-manual/sql-statements/Cluster-Management-Statements/ALTER-SYSTEM-DROP-BROKER.md: ########## @@ -25,42 +25,48 @@ specific language governing permissions and limitations under the License. --> -## ALTER-SYSTEM-DROP-BROKER +## Description -### Name +This statement is used to delete BROKER nodes. (For administrators only) -ALTER SYSTEM DROP BROKER +## Syntax -### Description - -This statement is to delete the BROKER node, (administrator only) - -grammar: +1. Drop all Brokers +```sql +ALTER SYSTEM DROP ALL BROKER broker_name; +``` +2. Drop one or more Broker nodes ```sql --- Delete all brokers -ALTER SYSTEM DROP ALL BROKER broker_name --- Delete a Broker node -ALTER SYSTEM DROP BROKER broker_name "host:port"[,"host:port"...]; +ALTER SYSTEM DROP BROKER broker_name "<host1>:<ipc_port>"[,"<host2>:<ipc_port>", ...]; ``` +## Required Parameters -### Example +**1. broker_name** +The name of the broker process to be deleted. -1. Delete all brokers +**2. host** +The IP of the node where the broker process to be deleted is located. If FQDN is enabled, use the FQDN of the node. - ```sql - ALTER SYSTEM DROP ALL BROKER broker_name - ``` +**3. ipc_port** +The PORT of the node where the broker process to be deleted is located, and the default value of this port is 8000. -2. Delete a Broker node +## Output 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
