Fix build after logical conflict (#14791)

alamb · web-flow · commit 28c97fcf3dfc · 2025-02-20T04:37:51.000-08:00
diff --git a/datafusion/functions/src/core/getfield.rs b/datafusion/functions/src/core/getfield.rs
@@ -236,11 +236,11 @@ impl ScalarUDFImpl for GetFieldFunc {
 
         match (array.data_type(), name) {
             (DataType::Map(_, _), ScalarValue::List(arr)) => {
-                let key_array: Arc<dyn Array> = Arc::new((**arr).clone());
+                let key_array: Arc<dyn Array> = arr;
                 process_map_array(array, key_array)
             }
             (DataType::Map(_, _), ScalarValue::Struct(arr)) => {
-                process_map_array(array, Arc::clone(arr) as Arc<dyn Array>)
+                process_map_array(array, arr as Arc<dyn Array>)
             }
             (DataType::Map(_, _), other) => {
                 let data_type = other.data_type();

Original file line number	Diff line number	Diff line change
`@@ -236,11 +236,11 @@ impl ScalarUDFImpl for GetFieldFunc {`
`236`	`236`
`237`	`237`	`match (array.data_type(), name) {`
`238`	`238`	`(DataType::Map(_, _), ScalarValue::List(arr)) => {`
`239`		`- let key_array: Arc<dyn Array> = Arc::new((**arr).clone());`
	`239`	`+ let key_array: Arc<dyn Array> = arr;`
`240`	`240`	`process_map_array(array, key_array)`
`241`	`241`	`}`
`242`	`242`	`(DataType::Map(_, _), ScalarValue::Struct(arr)) => {`
`243`		`- process_map_array(array, Arc::clone(arr) as Arc<dyn Array>)`
	`243`	`+ process_map_array(array, arr as Arc<dyn Array>)`
`244`	`244`	`}`
`245`	`245`	`(DataType::Map(_, _), other) => {`
`246`	`246`	`let data_type = other.data_type();`