fix[array]: fill_null don't run utf8 or binary

Signed-off-by: Joe Isaacs <[email protected]>

Author: joseph-isaacs

fuzz/src/array/mod.rs

@@ -381,8 +381,8 @@ fn actions_for_dtype(dtype: &DType) -> HashSet<ActionType> {
             [Compress, Slice, Take, Filter, MinMax, Mask, ScalarAt].into()
         }
         DType::Utf8(_) | DType::Binary(_) => {
-            // Utf8/Binary supports everything except Sum
-            // Actions: Compress, Slice, Take, SearchSorted, Filter, Compare, Cast, MinMax, FillNull, Mask, ScalarAt
+            // Utf8/Binary supports everything except Sum and FillNull
+            // Actions: Compress, Slice, Take, SearchSorted, Filter, Compare, Cast, MinMax, Mask, ScalarAt
             [
                 Compress,
                 Slice,
@@ -392,7 +392,6 @@ fn actions_for_dtype(dtype: &DType) -> HashSet<ActionType> {
                 Compare,
                 Cast,
                 MinMax,
-                FillNull,
                 Mask,
                 ScalarAt,
             ]

vortex-array/src/compute/fill_null.rs

@@ -108,6 +108,7 @@ impl ComputeFnVTable for FillNull {
             return Ok(fill_null(canonical_arr.as_ref(), fill_value)?.into());
         }
 
+        // TODO(joe): fill_null not implemented for DType Binary
         vortex_bail!("fill null not implemented for DType {}", array.dtype())
     }