morrySnow commented on code in PR #1884: URL: https://github.com/apache/doris-website/pull/1884#discussion_r1924790846
########## docs/sql-manual/sql-statements/Cluster-Management-Statements/ALTER-SYSTEM-ADD-BROKER.md: ########## @@ -25,38 +25,45 @@ 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> "<host>:<ipc_port>" [, "<host>:<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) Review Comment: ```suggestion This statement is used to delete BROKER nodes. ``` ########## docs/sql-manual/sql-statements/Cluster-Management-Statements/ALTER-SYSTEM-ADD-BROKER.md: ########## @@ -25,38 +25,45 @@ 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>** -```sql -ALTER SYSTEM ADD BROKER broker_name "broker_host1:broker_ipc_port1","broker_host2:broker_ipc_port2",...; -``` +The name given to the added broker process. It is recommended to keep the broker_name consistent within the same cluster. + +**2. < host>** Review Comment: ```suggestion **2. <host>** ``` ########## 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> "<host>:<ipc_port>"[, "<host>:<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; +``` +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 + +**1. <broker_name>** -### Example +The name of the broker process to be deleted. -1. Delete all brokers +**2. < host>** Review Comment: ```suggestion **2. <host>** ``` ########## docs/sql-manual/sql-statements/Cluster-Management-Statements/ALTER-SYSTEM-ADD-BROKER.md: ########## @@ -25,38 +25,45 @@ 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!) Review Comment: ```suggestion This statement is used to add one or more BROKER nodes. ``` -- 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
