clippy

Author: davidhewitt

arrow-select/src/concat.rs

@@ -132,7 +132,7 @@ fn merge_concat_byte_dictionaries<K: ArrowDictionaryKeyType>(
     dictionaries: &[&DictionaryArray<K>],
     output_len: usize,
 ) -> Result<ArrayRef, ArrowError> {
-    let merged = merge_dictionary_values(&dictionaries, None)?;
+    let merged = merge_dictionary_values(dictionaries, None)?;
 
     // Recompute keys
     let mut key_values = Vec::with_capacity(output_len);
@@ -341,7 +341,7 @@ where
 
 macro_rules! dict_helper {
     ($t:ty, $value_type:expr, $arrays:expr) => {
-        return Ok(concat_dictionaries::<$t>($value_type.as_ref(), $arrays)?)
+        concat_dictionaries::<$t>($value_type.as_ref(), $arrays)
     };
 }
 

arrow-select/src/interleave.rs

@@ -249,7 +249,7 @@ fn merge_interleave_byte_dictionaries<K: ArrowDictionaryKeyType>(
         })
         .collect();
 
-    let merged = merge_dictionary_values(&dictionaries, Some(&masks))?;
+    let merged = merge_dictionary_values(dictionaries, Some(&masks))?;
 
     // Recompute keys
     let mut keys = PrimitiveBuilder::<K>::with_capacity(indices.len());