From 21e9b980faade891cb6601535673ef44977c51a9 Mon Sep 17 00:00:00 2001 From: TCeason <33082201+TCeason@users.noreply.github.com> Date: Thu, 10 Oct 2024 11:59:06 +0800 Subject: [PATCH] chore(query): SHOW/DESC is not DDL (#16583) SHOW/DESC is not DDL Data definition language is a syntax for creating and modifying database objects such as tables, indices, and users. SHOW / DESC is not ddl. --- .../service/src/interpreters/interpreter_connection_show.rs | 2 +- .../service/src/interpreters/interpreter_data_mask_desc.rs | 2 +- .../src/interpreters/interpreter_database_show_create.rs | 2 +- .../service/src/interpreters/interpreter_file_format_show.rs | 2 +- .../src/interpreters/interpreter_network_policies_show.rs | 2 +- .../service/src/interpreters/interpreter_network_policy_desc.rs | 2 +- .../service/src/interpreters/interpreter_notification_desc.rs | 2 +- .../src/interpreters/interpreter_password_policy_desc.rs | 2 +- src/query/service/src/interpreters/interpreter_role_show.rs | 2 +- .../service/src/interpreters/interpreter_table_describe.rs | 2 +- .../service/src/interpreters/interpreter_table_show_create.rs | 2 +- src/query/service/src/interpreters/interpreter_task_describe.rs | 2 +- src/query/service/src/interpreters/interpreter_tasks_show.rs | 2 +- src/query/service/src/interpreters/interpreter_view_describe.rs | 2 +- 14 files changed, 14 insertions(+), 14 deletions(-) diff --git a/src/query/service/src/interpreters/interpreter_connection_show.rs b/src/query/service/src/interpreters/interpreter_connection_show.rs index e5d98727acf9..3240a9734ded 100644 --- a/src/query/service/src/interpreters/interpreter_connection_show.rs +++ b/src/query/service/src/interpreters/interpreter_connection_show.rs @@ -45,7 +45,7 @@ impl Interpreter for ShowConnectionsInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[fastrace::trace] diff --git a/src/query/service/src/interpreters/interpreter_data_mask_desc.rs b/src/query/service/src/interpreters/interpreter_data_mask_desc.rs index d048c18805fa..b62ee734f676 100644 --- a/src/query/service/src/interpreters/interpreter_data_mask_desc.rs +++ b/src/query/service/src/interpreters/interpreter_data_mask_desc.rs @@ -49,7 +49,7 @@ impl Interpreter for DescDataMaskInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[async_backtrace::framed] diff --git a/src/query/service/src/interpreters/interpreter_database_show_create.rs b/src/query/service/src/interpreters/interpreter_database_show_create.rs index f5385f9aa31c..1aebf8a6f779 100644 --- a/src/query/service/src/interpreters/interpreter_database_show_create.rs +++ b/src/query/service/src/interpreters/interpreter_database_show_create.rs @@ -46,7 +46,7 @@ impl Interpreter for ShowCreateDatabaseInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[async_backtrace::framed] diff --git a/src/query/service/src/interpreters/interpreter_file_format_show.rs b/src/query/service/src/interpreters/interpreter_file_format_show.rs index 3ff555df851e..948a49cf0107 100644 --- a/src/query/service/src/interpreters/interpreter_file_format_show.rs +++ b/src/query/service/src/interpreters/interpreter_file_format_show.rs @@ -44,7 +44,7 @@ impl Interpreter for ShowFileFormatsInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[fastrace::trace] diff --git a/src/query/service/src/interpreters/interpreter_network_policies_show.rs b/src/query/service/src/interpreters/interpreter_network_policies_show.rs index 63310662d7c5..edd641f3e032 100644 --- a/src/query/service/src/interpreters/interpreter_network_policies_show.rs +++ b/src/query/service/src/interpreters/interpreter_network_policies_show.rs @@ -43,7 +43,7 @@ impl Interpreter for ShowNetworkPoliciesInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[async_backtrace::framed] diff --git a/src/query/service/src/interpreters/interpreter_network_policy_desc.rs b/src/query/service/src/interpreters/interpreter_network_policy_desc.rs index a19aea3c27bb..9a5bdc526731 100644 --- a/src/query/service/src/interpreters/interpreter_network_policy_desc.rs +++ b/src/query/service/src/interpreters/interpreter_network_policy_desc.rs @@ -45,7 +45,7 @@ impl Interpreter for DescNetworkPolicyInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[async_backtrace::framed] diff --git a/src/query/service/src/interpreters/interpreter_notification_desc.rs b/src/query/service/src/interpreters/interpreter_notification_desc.rs index edffb26c6667..ff54e0736157 100644 --- a/src/query/service/src/interpreters/interpreter_notification_desc.rs +++ b/src/query/service/src/interpreters/interpreter_notification_desc.rs @@ -57,7 +57,7 @@ impl Interpreter for DescNotificationInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[fastrace::trace] diff --git a/src/query/service/src/interpreters/interpreter_password_policy_desc.rs b/src/query/service/src/interpreters/interpreter_password_policy_desc.rs index 951b639b01c2..e0e75fc96e5a 100644 --- a/src/query/service/src/interpreters/interpreter_password_policy_desc.rs +++ b/src/query/service/src/interpreters/interpreter_password_policy_desc.rs @@ -55,7 +55,7 @@ impl Interpreter for DescPasswordPolicyInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[async_backtrace::framed] diff --git a/src/query/service/src/interpreters/interpreter_role_show.rs b/src/query/service/src/interpreters/interpreter_role_show.rs index fa32adae2b25..cc181683dfcd 100644 --- a/src/query/service/src/interpreters/interpreter_role_show.rs +++ b/src/query/service/src/interpreters/interpreter_role_show.rs @@ -46,7 +46,7 @@ impl Interpreter for ShowRolesInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[fastrace::trace] diff --git a/src/query/service/src/interpreters/interpreter_table_describe.rs b/src/query/service/src/interpreters/interpreter_table_describe.rs index 764ccc10f68b..c0cdb536a8af 100644 --- a/src/query/service/src/interpreters/interpreter_table_describe.rs +++ b/src/query/service/src/interpreters/interpreter_table_describe.rs @@ -49,7 +49,7 @@ impl Interpreter for DescribeTableInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[async_backtrace::framed] diff --git a/src/query/service/src/interpreters/interpreter_table_show_create.rs b/src/query/service/src/interpreters/interpreter_table_show_create.rs index beabddf4d54e..8021e1d6f2ac 100644 --- a/src/query/service/src/interpreters/interpreter_table_show_create.rs +++ b/src/query/service/src/interpreters/interpreter_table_show_create.rs @@ -67,7 +67,7 @@ impl Interpreter for ShowCreateTableInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[async_backtrace::framed] diff --git a/src/query/service/src/interpreters/interpreter_task_describe.rs b/src/query/service/src/interpreters/interpreter_task_describe.rs index e6b7599ff866..1625954f60d9 100644 --- a/src/query/service/src/interpreters/interpreter_task_describe.rs +++ b/src/query/service/src/interpreters/interpreter_task_describe.rs @@ -58,7 +58,7 @@ impl Interpreter for DescribeTaskInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[fastrace::trace] diff --git a/src/query/service/src/interpreters/interpreter_tasks_show.rs b/src/query/service/src/interpreters/interpreter_tasks_show.rs index c65f2f24d620..3d9fdc299b37 100644 --- a/src/query/service/src/interpreters/interpreter_tasks_show.rs +++ b/src/query/service/src/interpreters/interpreter_tasks_show.rs @@ -69,7 +69,7 @@ impl Interpreter for ShowTasksInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[fastrace::trace] diff --git a/src/query/service/src/interpreters/interpreter_view_describe.rs b/src/query/service/src/interpreters/interpreter_view_describe.rs index d8ed87d2a04a..f65491fecc91 100644 --- a/src/query/service/src/interpreters/interpreter_view_describe.rs +++ b/src/query/service/src/interpreters/interpreter_view_describe.rs @@ -50,7 +50,7 @@ impl Interpreter for DescribeViewInterpreter { } fn is_ddl(&self) -> bool { - true + false } #[async_backtrace::framed]