morrySnow commented on code in PR #2192:
URL: https://github.com/apache/doris-website/pull/2192#discussion_r1997899189
##########
docs/sql-manual/sql-statements/cluster-management/instance-management/SET-FRONTEND-CONFIG.md:
##########
@@ -32,16 +32,15 @@ under the License.
This statement is used to set the configuration items of the cluster
(currently only supports setting FE configuration items).
-The configurable items can be viewed using the `SHOW FRONTEND CONFIG;` command.
-
-Syntax:
+## Syntax:
```sql
-ADMIN SET FRONTEND CONFIG ("key" = "value") [ALL];
--- or
-ADMIN SET ALL FRONTENDS CONFIG ("key" = "value");
+ADMIN SET [ALL FRONTENDS | FRONTEND] CONFIG ("<fe_config_key>" =
"<fe_config_value>")
Review Comment:
可以一次修改多个配置吗？如果可以的话，需要这样
```suggestion
ADMIN SET [ALL FRONTENDS | FRONTEND] CONFIG ("<fe_config_key>" =
"<fe_config_value>" [, ... ])
```
##########
docs/sql-manual/sql-statements/cluster-management/instance-management/SET-FRONTEND-CONFIG.md:
##########
@@ -32,16 +32,15 @@ under the License.
This statement is used to set the configuration items of the cluster
(currently only supports setting FE configuration items).
-The configurable items can be viewed using the `SHOW FRONTEND CONFIG;` command.
-
-Syntax:
+## Syntax:
```sql
-ADMIN SET FRONTEND CONFIG ("key" = "value") [ALL];
--- or
-ADMIN SET ALL FRONTENDS CONFIG ("key" = "value");
+ADMIN SET [ALL FRONTENDS | FRONTEND] CONFIG ("<fe_config_key>" =
"<fe_config_value>")
Review Comment:
```suggestion
ADMIN SET {ALL FRONTENDS | FRONTEND} CONFIG ("<fe_config_key>" =
"<fe_config_value>")
```
##########
docs/sql-manual/sql-statements/cluster-management/instance-management/SET-FRONTEND-CONFIG.md:
##########
@@ -32,16 +32,15 @@ under the License.
This statement is used to set the configuration items of the cluster
(currently only supports setting FE configuration items).
-The configurable items can be viewed using the `SHOW FRONTEND CONFIG;` command.
-
-Syntax:
+## Syntax:
```sql
-ADMIN SET FRONTEND CONFIG ("key" = "value") [ALL];
--- or
-ADMIN SET ALL FRONTENDS CONFIG ("key" = "value");
+ADMIN SET [ALL FRONTENDS | FRONTEND] CONFIG ("<fe_config_key>" =
"<fe_config_value>")
```
+## Optional Parameters
+`fe_config_key` and `fe_config_value` can be viewed and modified by `SHOW
FRONTEND CONFIG;` command
Review Comment:
如果是参考其他的语句，最好加上超链接
##########
docs/sql-manual/sql-statements/cluster-management/instance-management/SET-FRONTEND-CONFIG.md:
##########
@@ -32,16 +32,15 @@ under the License.
This statement is used to set the configuration items of the cluster
(currently only supports setting FE configuration items).
-The configurable items can be viewed using the `SHOW FRONTEND CONFIG;` command.
-
-Syntax:
+## Syntax:
```sql
-ADMIN SET FRONTEND CONFIG ("key" = "value") [ALL];
--- or
-ADMIN SET ALL FRONTENDS CONFIG ("key" = "value");
+ADMIN SET [ALL FRONTENDS | FRONTEND] CONFIG ("<fe_config_key>" =
"<fe_config_value>")
```
+## Optional Parameters
+`fe_config_key` and `fe_config_value` can be viewed and modified by `SHOW
FRONTEND CONFIG;` command
Review Comment:
这里的 modified 怎么理解？show 应该只能查看不能修改？
--
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
