Merge pull request #10528 from TCeason/ISSUE-10562

feat(query): system.tables add engine_full column

Author: BohuTANG

src/query/service/tests/it/servers/http/http_query_handlers.rs

@@ -126,7 +126,7 @@ async fn test_simple_sql() -> Result<()> {
     assert_eq!(result.state, ExecuteStateKind::Succeeded, "{:?}", result);
     assert_eq!(result.next_uri, Some(final_uri.clone()), "{:?}", result);
     assert_eq!(result.data.len(), 10, "{:?}", result);
-    assert_eq!(result.schema.len(), 11, "{:?}", result);
+    assert_eq!(result.schema.len(), 12, "{:?}", result);
 
     // get state
     let uri = make_state_uri(query_id);

src/query/service/tests/it/storages/testdata/columns_table.txt

@@ -92,6 +92,8 @@ DB.Table: 'system'.'columns', Table: columns-table_id:1, ver:0, Engine: SystemCo
 | "engine"                        | "information_schema" | "tables"              | "String"           | "VARCHAR"           | ""       | ""       | "NO"     | ""       |
 | "engine"                        | "system"             | "tables"              | "String"           | "VARCHAR"           | ""       | ""       | "NO"     | ""       |
 | "engine"                        | "system"             | "tables_with_history" | "String"           | "VARCHAR"           | ""       | ""       | "NO"     | ""       |
+| "engine_full"                   | "system"             | "tables"              | "String"           | "VARCHAR"           | ""       | ""       | "NO"     | ""       |
+| "engine_full"                   | "system"             | "tables_with_history" | "String"           | "VARCHAR"           | ""       | ""       | "NO"     | ""       |
 | "entry"                         | "system"             | "tracing"             | "String"           | "VARCHAR"           | ""       | ""       | "NO"     | ""       |
 | "event_date"                    | "system"             | "query_log"           | "Date"             | "DATE"              | ""       | ""       | "NO"     | ""       |
 | "event_time"                    | "system"             | "query_log"           | "Timestamp"        | "TIMESTAMP"         | ""       | ""       | "NO"     | ""       |

src/query/storages/system/src/tables_table.rs

@@ -136,6 +136,7 @@ where TablesTable<T>: HistoryAware
             .iter()
             .map(|v| v.engine().as_bytes().to_vec())
             .collect();
+        let engines_full: Vec<Vec<u8>> = engines.clone();
         let created_owns: Vec<String> = database_tables
             .iter()
             .map(|v| {
@@ -177,6 +178,7 @@ where TablesTable<T>: HistoryAware
             StringType::from_data(databases),
             StringType::from_data(names),
             StringType::from_data(engines),
+            StringType::from_data(engines_full),
             StringType::from_data(cluster_bys),
             StringType::from_data(created_owns),
             StringType::from_data(dropped_owns),
@@ -197,6 +199,7 @@ where TablesTable<T>: HistoryAware
             TableField::new("database", TableDataType::String),
             TableField::new("name", TableDataType::String),
             TableField::new("engine", TableDataType::String),
+            TableField::new("engine_full", TableDataType::String),
             TableField::new("cluster_by", TableDataType::String),
             TableField::new("created_on", TableDataType::String),
             TableField::new("dropped_on", TableDataType::String),