fix one safety warning and ignore some slow tests

core/Cargo.toml

@@ -97,6 +97,7 @@ twox-hash = "1.6.3"
 
 [features]
 default = []
+nightly = []
 
 [profile.release]
 debug = true

core/src/execution/datafusion/expressions/cast.rs

@@ -1646,6 +1646,7 @@ mod tests {
     use super::*;
 
     #[test]
+    #[cfg_attr(miri, ignore)] // test takes too long with miri
     fn timestamp_parser_test() {
         // write for all formats
         assert_eq!(
@@ -1683,6 +1684,7 @@ mod tests {
     }
 
     #[test]
+    #[cfg_attr(miri, ignore)] // test takes too long with miri
     fn test_cast_string_to_timestamp() {
         let array: ArrayRef = Arc::new(StringArray::from(vec![
             Some("2020-01-01T12:34:56.123456"),

core/src/execution/datafusion/expressions/xxhash64.rs

@@ -162,6 +162,7 @@ mod test {
     use twox_hash::XxHash64;
 
     #[test]
+    #[cfg_attr(miri, ignore)] // test takes too long with miri
     fn test_xxhash64_random() {
         let mut rng = rand::thread_rng();
         for len in 0..128 {

core/src/execution/datafusion/shuffle_writer.rs

@@ -1447,6 +1447,7 @@ mod test {
     }
 
     #[test]
+    #[cfg_attr(miri, ignore)] // miri can't call foreign function `ZSTD_createCCtx`
     fn test_insert_larger_batch() {
         let schema = Arc::new(Schema::new(vec![Field::new("a", DataType::Utf8, true)]));
         let mut b = StringBuilder::new();

core/src/execution/kernels/temporal.rs

@@ -824,6 +824,7 @@ mod tests {
     use std::sync::Arc;
 
     #[test]
+    #[cfg_attr(miri, ignore)] // test takes too long with miri
     fn test_date_trunc() {
         let size = 1000;
         let mut vec: Vec<i32> = Vec::with_capacity(size);
@@ -962,6 +963,7 @@ mod tests {
     }
 
     #[test]
+    #[cfg_attr(miri, ignore)] // test takes too long with miri
     fn test_timestamp_trunc() {
         let size = 1000;
         let mut vec: Vec<i64> = Vec::with_capacity(size);
@@ -998,6 +1000,7 @@ mod tests {
     }
 
     #[test]
+    #[cfg_attr(miri, ignore)] // test takes too long with miri
     // This test only verifies that the various input array types work. Actually correctness to
     // ensure this produces the same results as spark is verified in the JVM tests
     fn test_timestamp_trunc_array_fmt_dyn() {

core/src/execution/sort.rs

@@ -165,7 +165,8 @@ where
                         // because they are defined as Vec<Vec<T>>
                         ptr::copy_nonoverlapping(
                             bucket.as_ptr(),
-                            self.get_unchecked_mut(pos),
+                            //self.get_unchecked_mut(pos),
+                            self.as_mut_ptr().add(pos),
                             bucket.len(),
                         );
                     }