chore: refactor

Author: gandronchik

datafusion/core/src/execution/context.rs

@@ -2310,8 +2310,8 @@ mod tests {
                 let start_number = start.unwrap();
                 batch_sizes.push(1);
 
-                string_builder.append_value("test".to_string()).unwrap();
-                int_builder.append_value(start_number.clone()).unwrap();
+                string_builder.append_value("test").unwrap();
+                int_builder.append_value(start_number).unwrap();
             }
 
             let mut fields = Vec::new();

datafusion/core/src/sql/planner.rs

@@ -723,7 +723,7 @@ impl<'a, S: ContextProvider> SqlToRel<'a, S> {
 
                                 let alias = alias.unwrap();
 
-                                if alias.columns.len() > 0 {
+                                if !alias.columns.is_empty() {
                                     select_exprs = select_exprs
                                         .iter()
                                         .enumerate()
@@ -743,7 +743,7 @@ impl<'a, S: ContextProvider> SqlToRel<'a, S> {
                                 return project_with_alias(
                                     udtf_plan,
                                     select_exprs,
-                                    Some(alias.name.value.to_string()),
+                                    Some(alias.name.value),
                                 );
                             }