You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@datafusion.apache.org by ag...@apache.org on 2025/08/19 21:46:17 UTC
(datafusion-comet) branch main updated: trivial: remove unnecessary clone()s (#2066)
This is an automated email from the ASF dual-hosted git repository.
agrove pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/datafusion-comet.git
The following commit(s) were added to refs/heads/main by this push:
new 13b9320e8 trivial: remove unnecessary clone()s (#2066)
13b9320e8 is described below
commit 13b9320e8320cab85a5dcfd2f3355907a058918e
Author: Šimon Lukašík <lu...@gmail.com>
AuthorDate: Tue Aug 19 23:46:13 2025 +0200
trivial: remove unnecessary clone()s (#2066)
---
native/core/src/execution/planner.rs | 8 +++-----
1 file changed, 3 insertions(+), 5 deletions(-)
diff --git a/native/core/src/execution/planner.rs b/native/core/src/execution/planner.rs
index 71bb0c748..0c3d345c8 100644
--- a/native/core/src/execution/planner.rs
+++ b/native/core/src/execution/planner.rs
@@ -3388,7 +3388,7 @@ mod tests {
// generate test data in the temp folder
let tmp_dir = TempDir::new()?;
- let test_path = tmp_dir.path().to_str().unwrap().to_string();
+ let test_path = tmp_dir.path().to_str().unwrap();
let plan = session_ctx
.sql(test_data_query)
@@ -3397,13 +3397,11 @@ mod tests {
.await?;
// Write a parquet file into temp folder
- session_ctx
- .write_parquet(plan, test_path.clone(), None)
- .await?;
+ session_ctx.write_parquet(plan, test_path, None).await?;
// Register all parquet with temp data as file groups
let mut file_groups: Vec<FileGroup> = vec![];
- for entry in std::fs::read_dir(&test_path)? {
+ for entry in std::fs::read_dir(test_path)? {
let entry = entry?;
let path = entry.path();
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@datafusion.apache.org
For additional commands, e-mail: commits-help@datafusion.apache.org