more fixes

Signed-off-by: Adam Gutglick <adam@spiraldb.com>

Author: AdamGS

vortex-datafusion/src/v2/source.rs

@@ -669,6 +669,6 @@ fn estimate_to_df_precision(est: Option<&Precision<u64>>) -> DFPrecision<usize>
         Some(Precision::Inexact(v)) => {
             DFPrecision::Inexact(usize::try_from(*v).unwrap_or(usize::MAX))
         }
-        None => DFPrecision::Absent,
+        Some(Precision::Absent) | None => DFPrecision::Absent,
     }
 }

vortex-duckdb/src/datasource.rs

@@ -608,7 +608,7 @@ impl<T: DataSourceTableFunction> TableFunction for T {
                 // Post-filter estimate is always a heuristic.
                 Cardinality::Estimate(postfilter_cardinality(v, has_non_optional_filter))
             }
-            None => Cardinality::Unknown,
+            Some(Precision::Absent) | None => Cardinality::Unknown,
         }
     }
 

vortex-ffi/src/scan.rs

@@ -204,7 +204,7 @@ fn write_estimate<T: Into<u64>>(estimate: Option<Precision<T>>, out: &mut vx_est
             out.r#type = vx_estimate_type::VX_ESTIMATE_INEXACT;
             out.estimate = value.into();
         }
-        None => {
+        Some(Precision::Absent) | None => {
             out.r#type = vx_estimate_type::VX_ESTIMATE_UNKNOWN;
         }
     }

vortex-jni/src/data_source.rs

@@ -204,7 +204,7 @@ pub extern "system" fn Java_dev_vortex_jni_NativeDataSource_rowCount(
         let (rows, cardinality) = match ds.inner.row_count() {
             Some(Precision::Exact(r)) => (r as jlong, 2),
             Some(Precision::Inexact(r)) => (r as jlong, 1),
-            None => (0, 0),
+            Some(Precision::Absent) | None => (0, 0),
         };
         out.set_region(env, 0, &[rows, cardinality])?;
         Ok(())

vortex-jni/src/scan.rs

@@ -216,7 +216,7 @@ pub extern "system" fn Java_dev_vortex_jni_NativeScan_partitionCount(
         let (rows, cardinality) = match scan.partition_count() {
             Some(Precision::Exact(v)) => (v as jlong, 2),
             Some(Precision::Inexact(v)) => (v as jlong, 1),
-            None => (0, 0),
+            Some(Precision::Absent) | None => (0, 0),
         };
         out.set_region(env, 0, &[rows, cardinality])?;
         Ok(())
@@ -286,7 +286,7 @@ pub extern "system" fn Java_dev_vortex_jni_NativePartition_rowCount(
         let (rows, cardinality) = match partition.row_count() {
             Some(Precision::Exact(v)) => (v as jlong, 2),
             Some(Precision::Inexact(v)) => (v as jlong, 1),
-            None => (0, 0),
+            Some(Precision::Absent) | None => (0, 0),
         };
         out.set_region(env, 0, &[rows, cardinality])?;
         Ok(())