fix: gc string view arrays in RepartitionExec

Fixes #20491

- Took the fix from `ExternalSorter` introduced in #14823
- If any `StringViewArray` columns are present in the repartitioned
  input, we gc them to reduce duplicate tracking of the same string view
  buffer.
- Fixes over-counting when there's a `RepartitionExec` above a partial
  agg on a `StringViewArray` column.

Author: Samyak2

datafusion/physical-plan/src/repartition/mod.rs

@@ -1359,6 +1359,26 @@ impl RepartitionExec {
 
             for res in partitioner.partition_iter(batch)? {
                 let (partition, batch) = res?;
+
+                let batch = {
+                    use arrow::array::{Array, StringViewArray};
+                    let mut new_columns = Vec::with_capacity(batch.num_columns());
+                    let mut mutated = false;
+                    for col in batch.columns() {
+                        if let Some(sv) = col.as_any().downcast_ref::<StringViewArray>() {
+                            new_columns.push(Arc::new(sv.gc()) as Arc<dyn Array>);
+                            mutated = true;
+                        } else {
+                            new_columns.push(Arc::clone(col));
+                        }
+                    }
+                    if mutated {
+                        RecordBatch::try_new(batch.schema(), new_columns)?
+                    } else {
+                        batch
+                    }
+                };
+
                 let size = batch.get_array_memory_size();
 
                 let timer = metrics.send_time[partition].timer();