Merge pull request #9217 from lichuang/issue_9162

fix: fix ConstColumn(String) cast to StringColumn bug

Author: BohuTANG

src/query/datavalues/src/columns/column.rs

@@ -31,6 +31,7 @@ use crate::TypeID;
 pub type ColumnRef = Arc<dyn Column>;
 pub trait Column: Send + Sync {
     fn as_any(&self) -> &dyn Any;
+
     /// Type of data that column contains. It's an underlying physical type:
     /// Int32 for Date, Int64 for Timestamp, so on.
     fn data_type_id(&self) -> TypeID {

src/query/datavalues/src/columns/const_/mod.rs

@@ -143,3 +143,15 @@ impl Column for ConstColumn {
         self.column.serialize(vec, 0);
     }
 }
+
+impl std::fmt::Debug for ConstColumn {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        write!(
+            f,
+            "ConstColumn \t typeid: {:?}\t len: {}\t data: [{:?}]",
+            self.data_type_id(),
+            self.len(),
+            self.column,
+        )
+    }
+}

src/query/functions/src/scalars/comparisons/comparison.rs

@@ -555,8 +555,16 @@ impl<T: StringSearchImpl> ComparisonExpression for ComparisonStringImpl<T> {
                 T::vector_const(lhs, &r, self.op)
             }
             false => {
-                let lhs: &StringColumn = Series::check_get(&lhs)?;
-                let rhs: &StringColumn = Series::check_get(&rhs)?;
+                let full_lhs = lhs.convert_full_column();
+                let lhs: &StringColumn = match Series::check_get(&lhs) {
+                    Ok(lhs) => lhs,
+                    Err(_) => Series::check_get(&full_lhs)?,
+                };
+                let full_rhs = rhs.convert_full_column();
+                let rhs: &StringColumn = match Series::check_get(&rhs) {
+                    Ok(lhs) => lhs,
+                    Err(_) => Series::check_get(&full_rhs)?,
+                };
                 T::vector_vector(lhs, rhs, self.op)
             }
         };