From a207bd5aca8549a7340cdcb4d373243fea4f0b92 Mon Sep 17 00:00:00 2001
From: Maxime Dion <maxdion@gmail.com>
Date: Mon, 9 Sep 2024 22:44:26 -0500
Subject: [PATCH] linter fixes

---
 src/datafusion/helpers.rs | 10 +++-------
 src/lib.rs                |  1 -
 2 files changed, 3 insertions(+), 8 deletions(-)

diff --git a/src/datafusion/helpers.rs b/src/datafusion/helpers.rs
index 0bebc30..4ed1bdb 100644
--- 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
index 425ef5e..bc0a5c7 100644
--- 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")