Re: [PR] statments for session query [doris-website]

2025-01-22 Thread via GitHub


KassieZ merged PR #1901:
URL: https://github.com/apache/doris-website/pull/1901


-- 
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



Re: [PR] statments for session query [doris-website]

2025-01-22 Thread via GitHub


wyx123654 commented on PR #1901:
URL: https://github.com/apache/doris-website/pull/1901#issuecomment-2608762782

   have revised


-- 
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



Re: [PR] statments for session query [doris-website]

2025-01-22 Thread via GitHub


morrySnow commented on code in PR #1901:
URL: https://github.com/apache/doris-website/pull/1901#discussion_r1925307443


##
docs/sql-manual/sql-statements/session/queries/CLEAN-ALL-QUERY-STATS.md:
##
@@ -24,46 +24,64 @@ specific language governing permissions and limitations
 under the License.
 -->
 
-
-
 ## Description
 
 This statement is used to clear query statistics
 
-grammar:
+## Syntax
 
 ```sql
-CLEAN [ALL| DATABASE | TABLE] QUERY STATS [[FOR db_name]|[FROM|IN] 
table_name]];
+CLEAN [ { ALL| DATABASE | TABLE } ] QUERY STATS [ { [ FOR ] | [ { 
FROM | IN } ] ]];
 ```
 
-Remarks:
+## Required Parameters
+
+**1. `ALL`**
+
+> Use ALL to clear all query statistics
+
+**2. `DATABASE`**
+
+> DATABASE clears statistics of a database

Review Comment:
   database 和 table 也按照 ALL 的方式改一下~



##
docs/sql-manual/sql-statements/session/queries/CLEAN-ALL-QUERY-STATS.md:
##
@@ -24,46 +24,64 @@ specific language governing permissions and limitations
 under the License.
 -->
 
-
-
 ## Description
 
 This statement is used to clear query statistics
 
-grammar:
+## Syntax
 
 ```sql
-CLEAN [ALL| DATABASE | TABLE] QUERY STATS [[FOR db_name]|[FROM|IN] 
table_name]];
+CLEAN [ { ALL| DATABASE | TABLE } ] QUERY STATS [ { [ FOR ] | [ { 
FROM | IN } ] ]];
 ```
 
-Remarks:
+## Required Parameters
+
+**1. `ALL`**
+
+> Use ALL to clear all query statistics
+
+**2. `DATABASE`**
+
+> DATABASE clears statistics of a database
+
+**3. `TABLE`**
+
+> TABLE Indicates that statistics of a table are cleared
+
+## Optional Parameters
+
+**1. ``**
 
-1. If ALL is specified, all query statistics are cleared, including database 
and table, admin privilege is needed
-2. If DATABASE is specified, the query statistics of the specified database 
are cleared, alter privilege for this database is needed
-3. If TABLE is specified, the query statistics of the specified table are 
cleared, alter privilege for this table is needed
+> If this parameter is set, the statistics of the corresponding database are 
cleared
 
-## Example
+**2. ``**
 
-1. Clear all statistics
-2. 
-```sql
-clean all query stats;
-```
+> If this parameter is set, the statistics of the corresponding table are 
cleared
 
-2. Clear the specified database statistics
 
-```sql
-clean database query stats for test_query_db;
-```
-3. Clear the specified table statistics
+## Access Control Requirements
 
-```sql
-clean table query stats from test_query_db.baseall;
-```
+The user who executes this SQL command must have at least the following 
permissions:
 
-## Keywords
+| Privilege| Object   | Notes |
+|:-|:-|:--|
+| ADMIN| ALL  | If ALL is specified, the ADMIN permission is 
required |

Review Comment:
   ```suggestion
   | ADMIN_PRIV| ALL  | If ALL is specified, the ADMIN permission 
is required |
   ```



##
docs/sql-manual/sql-statements/session/queries/KILL-QUERY.md:
##
@@ -24,62 +24,74 @@ specific language governing permissions and limitations
 under the License.
 -->
 
+## Description
 
+Each Doris connection runs in a separate thread. Use this statement to 
terminate the thread.
 
-In Doris, each connection runs in a separate thread. You can terminate a 
thread using the `KILL processlist_id`statement.
+## Syntax
 
-The `processlist_id` for the thread can be found in the Id column from the 
SHOW PROCESSLIST output. Or you can use the `SELECT CONNECTION_ID()` command to 
query the current connection id.
+```SQL
+KILL [CONNECTION] 
+```
 
-Syntax:
+## Varaint Syntax
 
-```sql
-KILL [CONNECTION] processlist_id
+```SQL
+KILL QUERY [ {  |  } ]
 ```
 
-## Kill query
+## Required Parameters
 
-You can also terminate the query command under execution based on the 
processlist_id or the query_id.
+**1. ``**
 
-Syntax:
+> Indicates the ID of the connection thread that needs to be killed
 
-```sql
-KILL QUERY processlist_id | query_id
-```
+**2. ``**
+
+> Indicates the ID of the query to be killed
+
+## Optional Parameters
+
+**1. `CONNECTION`**
+
+> Indicates whether the thread is currently connected
+
+## Usage Notes
 
-## Example
+- The thread process list identifier can be queried from the Id column output 
by the SHOW PROCESSLIST
 
-1. Check the current connection id.

Review Comment:
   kill query 不需要权限?



##
docs/sql-manual/sql-statements/session/queries/CLEAN-ALL-QUERY-STATS.md:
##
@@ -24,46 +24,64 @@ specific language governing permissions and limitations
 under the License.
 -->
 
-
-
 ## Description
 
 This statement is used to clear query statistics
 
-grammar:
+## Syntax
 
 ```sql
-CLEAN [ALL| DATABASE | TABLE] QUERY STATS [[FOR db_name]|[FROM|IN] 
table_name]];
+CLEAN [ { ALL| DATABASE | TABLE } ] QUERY STATS [ { [ FOR ] | [ { 
FROM | IN } ] ]];
 ```
 
-Remarks:
+## Required Parameters
+
+**1. `ALL`**
+
+> Use ALL to clear all query statistics
+
+**2. `DATABASE`**
+
+> DATABASE 

Re: [PR] statments for session query [doris-website]

2025-01-22 Thread via GitHub


wyx123654 commented on PR #1901:
URL: https://github.com/apache/doris-website/pull/1901#issuecomment-2607211982

   have revised


-- 
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



Re: [PR] statments for session query [doris-website]

2025-01-22 Thread via GitHub


morrySnow commented on code in PR #1901:
URL: https://github.com/apache/doris-website/pull/1901#discussion_r1925217370


##
docs/sql-manual/sql-statements/session/queries/CLEAN-ALL-QUERY-STATS.md:
##
@@ -24,46 +24,64 @@ specific language governing permissions and limitations
 under the License.
 -->
 
-
-
 ## Description
 
 This statement is used to clear query statistics
 
-grammar:
+## Syntax
 
 ```sql
-CLEAN [ALL| DATABASE | TABLE] QUERY STATS [[FOR db_name]|[FROM|IN] 
table_name]];
+CLEAN [ { ALL| DATABASE | TABLE } ] QUERY STATS [ { [ FOR ] | [ { 
FROM | IN } ] ]];
 ```
 
-Remarks:
+## Required Parameters
+
+**1. `ALL`**
+
+> ALL Clears all statistics

Review Comment:
   stats 是 
   ```suggestion
   > Use ALL to clear all query statistics
   ```



-- 
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