refactor: use from_epoch_days instead

Author: Sl1mb0

guests/python/src/conversion.rs

@@ -207,7 +207,7 @@ impl PythonType {
                     maybe_val
                         .map(|val| {
                             let days_since_epoch = val;
-                            let epoch = NaiveDate::from_ymd_opt(1970, 1, 1)
+                            let epoch = NaiveDate::from_epoch_days(0)
                                 .ok_or_else(|| exec_datafusion_err!("cannot create epoch date"))?;
                             let date = epoch + chrono::Duration::days(days_since_epoch as i64);
 
@@ -568,7 +568,7 @@ impl<'py> ArrayBuilder<'py> for Date32Builder {
                 )
             })?;
 
-        let epoch = NaiveDate::from_ymd_opt(1970, 1, 1)
+        let epoch = NaiveDate::from_epoch_days(0)
             .ok_or_else(|| exec_datafusion_err!("cannot create epoch date"))?;
 
         let days_since_epoch = date.signed_duration_since(epoch).num_days();

host/tests/integration_tests/python/types/date.rs

@@ -70,12 +70,12 @@ def foo(x: date) -> date:
             args: vec![ColumnarValue::Array(Arc::new(Date32Array::from_iter([
                 Some(0),
             ])))],
-            arg_fields: vec![Arc::new(Field::new("a1", DataType::Date32, true))],
+            arg_fields: vec![Arc::new(Field::new("time", DataType::Date32, true))],
             number_rows: 1,
             return_field: Arc::new(Field::new("r", DataType::Date32, true)),
         })
         .unwrap_err();
-    // The error message will contain the actual datetime, so we just check it contains the expected type info
+
     assert!(err.to_string().contains("expected `date` but got"));
     assert!(err.to_string().contains("of type `datetime`"));
 }

host/tests/integration_tests/python/types/time.rs

@@ -144,7 +144,7 @@ def foo(x: time) -> time:
             )),
         })
         .unwrap_err();
-    // The error message will contain the actual datetime, so we just check it contains the expected type info
+
     assert!(err.to_string().contains("expected `time` but got"));
     assert!(err.to_string().contains("of type `datetime`"));
 }

host/tests/integration_tests/python/types/timedelta.rs

@@ -137,7 +137,7 @@ def foo(x: timedelta) -> timedelta:
             )),
         })
         .unwrap_err();
-    // The error message will contain the actual datetime, so we just check it contains the expected type info
+
     assert!(err.to_string().contains("expected `timedelta` but got"));
     assert!(err.to_string().contains("of type `datetime`"));
 }