fix: ExistsTable should has_result_set. (#15431)

* fix: ExistsTable should has_result_set.

* ci: fix exists_tables.test

* refactor: let has_result_set = schema.is_empty().

Author: youngsofun

src/query/sql/src/planner/plans/plan.rs

@@ -479,50 +479,11 @@ impl Plan {
             Plan::ExecuteImmediate(plan) => plan.schema(),
             Plan::InsertMultiTable(plan) => plan.schema(),
 
-            other => {
-                debug_assert!(!other.has_result_set());
-                Arc::new(DataSchema::empty())
-            }
+            _ => Arc::new(DataSchema::empty()),
         }
     }
 
     pub fn has_result_set(&self) -> bool {
-        matches!(
-            self,
-            Plan::Query { .. }
-                | Plan::Explain { .. }
-                | Plan::ExplainAst { .. }
-                | Plan::ExplainSyntax { .. }
-                | Plan::ExplainAnalyze { .. }
-                | Plan::ShowCreateDatabase(_)
-                | Plan::ShowCreateTable(_)
-                | Plan::ShowCreateCatalog(_)
-                | Plan::ShowFileFormats(_)
-                | Plan::ShowRoles(_)
-                | Plan::DescShare(_)
-                | Plan::ShowShares(_)
-                | Plan::ShowShareEndpoint(_)
-                | Plan::ShowObjectGrantPrivileges(_)
-                | Plan::ShowGrantTenantsOfShare(_)
-                | Plan::DescribeTable(_)
-                | Plan::ShowGrants(_)
-                | Plan::Presign(_)
-                | Plan::VacuumTable(_)
-                | Plan::VacuumDropTable(_)
-                | Plan::VacuumTemporaryFiles(_)
-                | Plan::DescDatamaskPolicy(_)
-                | Plan::DescNetworkPolicy(_)
-                | Plan::ShowNetworkPolicies(_)
-                | Plan::DescPasswordPolicy(_)
-                | Plan::CopyIntoTable(_)
-                | Plan::CopyIntoLocation(_)
-                | Plan::ShowTasks(_)
-                | Plan::DescribeTask(_)
-                | Plan::DescConnection(_)
-                | Plan::ShowConnections(_)
-                | Plan::MergeInto(_)
-                | Plan::ExecuteImmediate(_)
-                | Plan::InsertMultiTable(_)
-        )
+        !self.schema().fields().is_empty()
     }
 }

tests/sqllogictests/suites/base/05_ddl/05_0023_exists_table.test

@@ -7,30 +7,39 @@ CREATE DATABASE db_05_0023_v2
 statement ok
 USE db_05_0023_v2
 
-statement ok
+query I
 EXISTS TABLE t
+----
+0
 
-statement ok
+query I
 EXISTS TABLE db_05_0023_v2.t
+----
+0
 
 statement ok
 CREATE TABLE t(c1 int) ENGINE = Fuse
 
-statement ok
+query I
 EXISTS TABLE t
+----
+1
 
-statement ok
+query I
 EXISTS TABLE db_05_0023_v2.t
+----
+1
 
 statement ok
 DROP TABLE t ALL
 
 statement ok
 EXISTS TABLE db_05_0023_v2.t
 
-statement ok
+query I
 EXISTS TABLE t
+----
+0
 
 statement ok
 DROP database db_05_0023_v2
-