Overtime events

Author: gatesn

fuzz/fuzz_targets/file_io.rs

@@ -42,7 +42,7 @@ fuzz_target!(|fuzz: FuzzFileAction| -> Corpus {
         let filtered = filter(&array_data, &mask).vortex_unwrap();
         projection_expr
             .clone()
-            .unwrap_or_else(|| root())
+            .unwrap_or_else(root)
             .evaluate(&Scope::new(filtered))
             .vortex_unwrap()
     };
@@ -69,7 +69,7 @@ fuzz_target!(|fuzz: FuzzFileAction| -> Corpus {
         .vortex_unwrap()
         .scan()
         .vortex_unwrap()
-        .with_projection(projection_expr.unwrap_or_else(|| root()))
+        .with_projection(projection_expr.unwrap_or_else(root))
         .with_some_filter(filter_expr)
         .into_array_iter(&*RUNTIME)
         .vortex_unwrap()

vortex-layout/src/gpu/layouts/struct_/reader.rs

@@ -136,11 +136,7 @@ impl GpuStructReader {
                     // expression by replacing any `$.a` with `$`.
                     return Partitioned::Single(
                         partitioned.partition_names[0].clone(),
-                        replace(
-                            expr.clone(),
-                            &col(partitioned.partition_names[0].clone()),
-                            root(),
-                        ),
+                        replace(expr, &col(partitioned.partition_names[0].clone()), root()),
                     );
                 }
 

vortex-layout/src/layouts/struct_/reader.rs

@@ -154,11 +154,7 @@ impl StructReader {
                     // expression by replacing any `$.a` with `$`.
                     return Partitioned::Single(
                         partitioned.partition_names[0].clone(),
-                        replace(
-                            expr.clone(),
-                            &col(partitioned.partition_names[0].clone()),
-                            root(),
-                        ),
+                        replace(expr, &col(partitioned.partition_names[0].clone()), root()),
                     );
                 }