linter fixes

datafusion-contrib · Sep 10, 2024 · a207bd5 · a207bd5
1 parent d987de9
commit a207bd5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 8 deletions.
diff --git a/src/datafusion/helpers.rs b/src/datafusion/helpers.rs
@@ -247,13 +247,9 @@ pub(crate) fn build_row_filter(
     let candidates: Vec<ZarrFilterCandidate> = predicates
         .into_iter()
         .flat_map(|expr| {
-            if let Ok(candidate) =
-                ZarrFilterCandidateBuilder::new(expr.clone(), file_schema).build()
-            {
-                candidate
-            } else {
-                None
-            }
+            ZarrFilterCandidateBuilder::new(expr.clone(), file_schema)
+                .build()
+                .unwrap_or_default()
         })
         .collect();
 

diff --git a/src/lib.rs b/src/lib.rs
@@ -26,7 +26,6 @@ mod tests {
     use object_store::local::LocalFileSystem;
     use std::path::PathBuf;
 
-    #[cfg(feature = "datafusion")]
     pub(crate) fn get_test_v2_data_path(zarr_store: String) -> PathBuf {
         PathBuf::from(env!("CARGO_MANIFEST_DIR"))
             .join("test-data/data/zarr/v2_data")