Merge pull request #10483 from andylokandy/desc

refactor(expr): combine TypeDeserializer and ColumnBuilder

Author: BohuTANG

Cargo.lock

@@ -93,8 +93,7 @@ impl PushDownInfo {
             if let RemoteExpr::<String>::ColumnRef { id, .. } = &order.0 {
                 // TODO: support sub column of nested type.
                 let field = schema.field_with_name(id).unwrap();
-                let data_type: DataType = field.data_type().into();
-                if !support(&data_type) {
+                if !support(&field.data_type().into()) {
                     return None;
                 }
 

src/query/catalog/src/plan/pushdown.rs

@@ -27,7 +27,6 @@ comfy-table = "6"
 dashmap = "5.4"
 educe = "0.4"
 enum-as-inner = "0.5"
-enum_dispatch = "0.3.8"
 ethnum = { version = "1.3", features = ["serde", "macros"] }
 futures = "0.3.24"
 hex = "0.4.3"

src/query/expression/Cargo.toml

@@ -452,9 +452,8 @@ impl DataBlock {
             let column = if !block_column_ids.contains(&column_id) {
                 let default_val = &default_vals[i];
                 let table_data_type = field.data_type();
-                let data_type: DataType = table_data_type.into();
                 BlockEntry {
-                    data_type,
+                    data_type: table_data_type.into(),
                     value: Value::Scalar(default_val.to_owned()),
                 }
             } else {