add DataType::Data32 for try_into_data_array

databendlabs · Oct 21, 2021 · b8b128e · b8b128e
1 parent 351fc51
commit b8b128e
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/common/datavalues/src/data_value_ops.rs b/common/datavalues/src/data_value_ops.rs
@@ -88,6 +88,9 @@ impl DataValue {
             DataType::Date16 => {
                 try_build_array! {PrimitiveArrayBuilder, u16, UInt16, values}
             }
+            DataType::Date32 => {
+                try_build_array! {PrimitiveArrayBuilder, i32, Int32, values}
+            }
             DataType::DateTime32(_) => {
                 try_build_array! {PrimitiveArrayBuilder, u32, UInt32, values}
             }

diff --git a/common/datavalues/src/macros.rs b/common/datavalues/src/macros.rs
@@ -186,10 +186,7 @@ macro_rules! try_build_array {
             match value {
                 DataValue::$SCALAR_TY(Some(v)) => builder.append_value(*v),
                 DataValue::$SCALAR_TY(None) => builder.append_null(),
-                dv => {
-                    eprintln!("DV is {:?}", dv);
-                    unreachable!()
-                }
+                _ => unreachable!(),
             }
         }
         Ok(builder.finish().into_series())