fix: Do not extract expression from cast to date (#977)

solves #976 by not transforming `Cast`s to `DATE`.

Author: omerhadari

crates/integrations/datafusion/src/physical_plan/expr_to_predicate.rs

@@ -17,6 +17,7 @@
 
 use std::vec;
 
+use datafusion::arrow::datatypes::DataType;
 use datafusion::logical_expr::{Expr, Operator};
 use datafusion::scalar::ScalarValue;
 use iceberg::expr::{BinaryExpression, Predicate, PredicateOperator, Reference, UnaryExpression};
@@ -119,7 +120,14 @@ fn to_iceberg_predicate(expr: &Expr) -> TransformedResult {
                 _ => TransformedResult::NotTransformed,
             }
         }
-        Expr::Cast(c) => to_iceberg_predicate(&c.expr),
+        Expr::Cast(c) => {
+            if c.data_type == DataType::Date32 || c.data_type == DataType::Date64 {
+                // Casts to date truncate the expression, we cannot simply extract it as it
+                // can create erroneous predicates.
+                return TransformedResult::NotTransformed;
+            }
+            to_iceberg_predicate(&c.expr)
+        }
         _ => TransformedResult::NotTransformed,
     }
 }
@@ -403,4 +411,11 @@ mod tests {
             Reference::new("ts").greater_than_or_equal_to(Datum::string("2023-01-05T00:00:00"));
         assert_eq!(predicate, expected_predicate);
     }
+
+    #[test]
+    fn test_predicate_conversion_with_date_cast() {
+        let sql = "ts >= date '2023-01-05T11:00:00'";
+        let predicate = convert_to_iceberg_predicate(sql);
+        assert_eq!(predicate, None);
+    }
 }