vortex_array/arrays/list/compute/

take.rs

// SPDX-License-Identifier: Apache-2.0
// SPDX-FileCopyrightText: Copyright the Vortex contributors

use itertools::Itertools as _;
use vortex_buffer::BufferMut;
use vortex_error::VortexExpect;
use vortex_error::VortexResult;
use vortex_error::vortex_ensure_eq;
use vortex_error::vortex_err;
use vortex_mask::Mask;

use crate::ArrayRef;
use crate::Columnar;
use crate::IntoArray;
use crate::array::ArrayView;
use crate::arrays::ConstantArray;
use crate::arrays::List;
use crate::arrays::ListArray;
use crate::arrays::PiecewiseSequence;
use crate::arrays::PiecewiseSequenceArray;
use crate::arrays::Primitive;
use crate::arrays::PrimitiveArray;
use crate::arrays::dict::TakeExecute;
use crate::arrays::list::ListArrayExt;
use crate::arrays::list::ListArraySlotsExt;
use crate::arrays::piecewise_sequence::constant_unsigned_usize;
use crate::arrays::piecewise_sequence::maybe_contiguous_slices;
use crate::arrays::primitive::PrimitiveArrayExt;
use crate::dtype::IntegerPType;
use crate::dtype::UnsignedPType;
use crate::executor::ExecutionCtx;
use crate::match_each_unsigned_integer_ptype;
use crate::match_smallest_offset_type;
use crate::validity::Validity;

// TODO(connor)[ListView]: Re-revert to the version where we simply convert to a `ListView` and call
// the `ListView::take` compute function once `ListView` is more stable.

impl TakeExecute for List {
    /// Take implementation for [`ListArray`].
    ///
    /// Unlike `ListView`, `ListArray` must rebuild the elements array to maintain its invariant
    /// that lists are stored contiguously and in-order (`offset[i+1] >= offset[i]`). Taking
    /// non-contiguous indices would violate this requirement.
    #[expect(clippy::cognitive_complexity)]
    fn take(
        array: ArrayView<'_, List>,
        indices: &ArrayRef,
        ctx: &mut ExecutionCtx,
    ) -> VortexResult<Option<ArrayRef>> {
        if let Some(piecewise_indices) = indices.as_opt::<PiecewiseSequence>()
            && let Some(taken) = take_slices(array, piecewise_indices, indices, ctx)?
        {
            return Ok(Some(taken));
        }

        let new_validity = array.validity()?.take(indices)?;
        let indices = indices.clone().execute::<PrimitiveArray>(ctx)?;
        let indices = indices.reinterpret_cast(indices.ptype().to_unsigned());
        let offsets = array.offsets().clone().execute::<PrimitiveArray>(ctx)?;
        let offsets = offsets.reinterpret_cast(offsets.ptype().to_unsigned());
        let validity_mask = new_validity.execute_mask(indices.len(), ctx)?;
        // This is an over-approximation of the total number of elements in the resulting array.
        let total_approx = array.elements().len().saturating_mul(indices.len());

        match_each_unsigned_integer_ptype!(offsets.ptype(), |O| {
            match_each_unsigned_integer_ptype!(indices.ptype(), |I| {
                match_smallest_offset_type!(total_approx, |OutOffset| {
                    take_with_piecewise_elements::<I, O, OutOffset>(
                        array,
                        offsets.as_view(),
                        indices.as_view(),
                        new_validity,
                        &validity_mask,
                    )
                    .map(Some)
                })
            })
        })
    }
}

fn take_with_piecewise_elements<I: IntegerPType, O: IntegerPType, OutOffset: IntegerPType>(
    array: ArrayView<'_, List>,
    offsets_array: ArrayView<'_, Primitive>,
    indices_array: ArrayView<'_, Primitive>,
    new_validity: Validity,
    validity_mask: &Mask,
) -> VortexResult<ArrayRef> {
    let offsets: &[O] = offsets_array.as_slice();
    let indices: &[I] = indices_array.as_slice();

    let offsets_capacity = indices
        .len()
        .checked_add(1)
        .ok_or_else(|| vortex_err!("List take offsets length overflow"))?;
    let mut new_offsets = BufferMut::<OutOffset>::with_capacity(offsets_capacity);
    let mut element_starts = BufferMut::<u64>::with_capacity(indices.len());
    let mut element_lengths = BufferMut::<u64>::with_capacity(indices.len());

    let mut current_offset = 0usize;
    new_offsets.push(OutOffset::zero());

    for (&data_idx, is_valid) in indices.iter().zip_eq(validity_mask.iter()) {
        if !is_valid {
            new_offsets.push(new_offset_value::<OutOffset>(current_offset));
            element_starts.push(0);
            element_lengths.push(0);
            continue;
        }

        let data_idx: usize = data_idx.as_();

        let start = offsets[data_idx];
        let stop = offsets[data_idx + 1];
        let start: usize = start.as_();
        let stop: usize = stop.as_();
        let length = stop - start;

        current_offset = current_offset
            .checked_add(length)
            .ok_or_else(|| vortex_err!("List take output elements length overflow"))?;
        new_offsets.push(new_offset_value::<OutOffset>(current_offset));
        element_starts.push(start as u64);
        element_lengths.push(length as u64);
    }

    let new_offsets = PrimitiveArray::new(new_offsets.freeze(), Validity::NonNullable).into_array();
    let multipliers = ConstantArray::new(1u64, element_starts.len()).into_array();

    // SAFETY: valid source rows contribute ranges derived from list offsets; null index/source
    // rows contribute zero-length placeholder ranges. `current_offset` is the sum of all generated
    // element lengths, and multiplier 1 preserves contiguous ranges.
    let element_indices = unsafe {
        PiecewiseSequenceArray::new_unchecked(
            element_starts.into_array(),
            element_lengths.into_array(),
            multipliers,
            current_offset,
        )
    };
    let new_elements = array.elements().take(element_indices.into_array())?;

    // SAFETY: offsets are rebuilt from the gathered element ranges and have one entry per output
    // row plus the leading zero; validity is produced by the usual take-validity path.
    Ok(unsafe { ListArray::new_unchecked(new_elements, new_offsets, new_validity) }.into_array())
}

fn take_slices(
    array: ArrayView<'_, List>,
    indices: ArrayView<'_, PiecewiseSequence>,
    indices_ref: &ArrayRef,
    ctx: &mut ExecutionCtx,
) -> VortexResult<Option<ArrayRef>> {
    let Some((starts, lengths)) = maybe_contiguous_slices(indices, ctx)? else {
        return Ok(None);
    };
    let data_validity = array
        .list_validity()
        .execute_mask(array.as_ref().len(), ctx)?;
    let offsets = array.offsets().clone().execute::<PrimitiveArray>(ctx)?;
    let offsets = offsets.reinterpret_cast(offsets.ptype().to_unsigned());
    let output_len = indices_ref.len();

    let taken = match lengths {
        Columnar::Constant(lengths) => {
            let length = constant_unsigned_usize(&lengths);
            take_slices_constant_start_dispatch(
                array,
                &starts,
                length,
                &offsets,
                indices_ref,
                output_len,
                &data_validity,
            )?
        }
        Columnar::Canonical(lengths) => {
            let lengths = lengths.into_primitive();
            take_slices_start_dispatch(
                array,
                &starts,
                &lengths,
                &offsets,
                indices_ref,
                output_len,
                &data_validity,
            )?
        }
    };
    Ok(Some(taken))
}

fn take_slices_constant_start_dispatch(
    array: ArrayView<'_, List>,
    starts: &PrimitiveArray,
    length: usize,
    offsets: &PrimitiveArray,
    indices_ref: &ArrayRef,
    output_len: usize,
    data_validity: &Mask,
) -> VortexResult<ArrayRef> {
    match_each_unsigned_integer_ptype!(starts.ptype(), |S| {
        take_slices_constant_offset_dispatch::<S>(
            array,
            starts,
            length,
            offsets,
            indices_ref,
            output_len,
            data_validity,
        )
    })
}

fn take_slices_constant_offset_dispatch<S>(
    array: ArrayView<'_, List>,
    starts: &PrimitiveArray,
    length: usize,
    offsets: &PrimitiveArray,
    indices_ref: &ArrayRef,
    output_len: usize,
    data_validity: &Mask,
) -> VortexResult<ArrayRef>
where
    S: UnsignedPType,
{
    match_each_unsigned_integer_ptype!(offsets.ptype(), |O| {
        take_slices_constant_length::<S, O>(
            array,
            starts.as_slice::<S>(),
            length,
            offsets.as_slice::<O>(),
            indices_ref,
            output_len,
            data_validity,
        )
    })
}

fn take_slices_start_dispatch(
    array: ArrayView<'_, List>,
    starts: &PrimitiveArray,
    lengths: &PrimitiveArray,
    offsets: &PrimitiveArray,
    indices_ref: &ArrayRef,
    output_len: usize,
    data_validity: &Mask,
) -> VortexResult<ArrayRef> {
    match_each_unsigned_integer_ptype!(starts.ptype(), |S| {
        take_slices_length_dispatch::<S>(
            array,
            starts,
            lengths,
            offsets,
            indices_ref,
            output_len,
            data_validity,
        )
    })
}

fn take_slices_length_dispatch<S>(
    array: ArrayView<'_, List>,
    starts: &PrimitiveArray,
    lengths: &PrimitiveArray,
    offsets: &PrimitiveArray,
    indices_ref: &ArrayRef,
    output_len: usize,
    data_validity: &Mask,
) -> VortexResult<ArrayRef>
where
    S: UnsignedPType,
{
    match_each_unsigned_integer_ptype!(lengths.ptype(), |L| {
        take_slices_offset_dispatch::<S, L>(
            array,
            starts,
            lengths,
            offsets,
            indices_ref,
            output_len,
            data_validity,
        )
    })
}

fn take_slices_offset_dispatch<S, L>(
    array: ArrayView<'_, List>,
    starts: &PrimitiveArray,
    lengths: &PrimitiveArray,
    offsets: &PrimitiveArray,
    indices_ref: &ArrayRef,
    output_len: usize,
    data_validity: &Mask,
) -> VortexResult<ArrayRef>
where
    S: UnsignedPType,
    L: UnsignedPType,
{
    match_each_unsigned_integer_ptype!(offsets.ptype(), |O| {
        take_slices_typed::<S, L, O>(
            array,
            starts.as_slice::<S>(),
            lengths.as_slice::<L>(),
            offsets.as_slice::<O>(),
            indices_ref,
            output_len,
            data_validity,
        )
    })
}

fn take_slices_constant_length<S, Offset>(
    array: ArrayView<'_, List>,
    starts: &[S],
    length: usize,
    offsets: &[Offset],
    indices_ref: &ArrayRef,
    output_len: usize,
    data_validity: &Mask,
) -> VortexResult<ArrayRef>
where
    S: UnsignedPType,
    Offset: UnsignedPType,
{
    let computed_len = starts
        .len()
        .checked_mul(length)
        .ok_or_else(|| vortex_err!("PiecewiseSequenceArray output length overflows usize"))?;
    vortex_ensure_eq!(
        computed_len,
        output_len,
        "PiecewiseSequenceArray expanded length does not match declared length",
    );
    let all_valid = data_validity.all_true();
    let total_elements = if all_valid {
        piecewise_list_elements_len_constant(offsets, starts, length)?
    } else {
        piecewise_list_elements_len_constant_validity(offsets, starts, length, data_validity)?
    };
    let validity = array.validity()?.take(indices_ref)?;

    match_smallest_offset_type!(total_elements, |OutOffset| {
        let gathered = if all_valid {
            gather_piecewise_list_constant_length::<S, Offset, OutOffset>(
                array.elements(),
                offsets,
                starts,
                length,
                output_len,
                total_elements,
            )?
        } else {
            gather_piecewise_list_constant_length_validity::<S, Offset, OutOffset>(
                array.elements(),
                offsets,
                starts,
                length,
                output_len,
                total_elements,
                data_validity,
            )?
        };

        // SAFETY: output offsets are rebuilt from valid monotonic source offsets; output elements
        // are exactly the gathered child ranges referenced by those offsets; validity has one bit
        // per output row.
        Ok(
            unsafe { ListArray::new_unchecked(gathered.elements, gathered.offsets, validity) }
                .into_array(),
        )
    })
}

fn take_slices_typed<S, L, Offset>(
    array: ArrayView<'_, List>,
    starts: &[S],
    lengths: &[L],
    offsets: &[Offset],
    indices_ref: &ArrayRef,
    output_len: usize,
    data_validity: &Mask,
) -> VortexResult<ArrayRef>
where
    S: UnsignedPType,
    L: UnsignedPType,
    Offset: UnsignedPType,
{
    let mut computed_len = 0usize;
    for &length in lengths {
        let length: usize = length.as_();
        computed_len = computed_len
            .checked_add(length)
            .ok_or_else(|| vortex_err!("PiecewiseSequenceArray output length overflows usize"))?;
    }
    vortex_ensure_eq!(
        computed_len,
        output_len,
        "PiecewiseSequenceArray expanded length does not match declared length",
    );
    let all_valid = data_validity.all_true();
    let total_elements = if all_valid {
        piecewise_list_elements_len(offsets, starts, lengths)?
    } else {
        piecewise_list_elements_len_validity(offsets, starts, lengths, data_validity)?
    };

    match_smallest_offset_type!(total_elements, |OutOffset| {
        let gathered = if all_valid {
            gather_piecewise_list::<S, L, Offset, OutOffset>(
                array.elements(),
                offsets,
                starts,
                lengths,
                output_len,
                total_elements,
            )?
        } else {
            gather_piecewise_list_validity::<S, L, Offset, OutOffset>(
                array.elements(),
                offsets,
                starts,
                lengths,
                output_len,
                total_elements,
                data_validity,
            )?
        };
        let validity = array.validity()?.take(indices_ref)?;

        // SAFETY: output offsets are rebuilt from valid monotonic source offsets; output elements
        // are exactly the gathered child ranges referenced by those offsets; validity has one bit
        // per output row.
        Ok(
            unsafe { ListArray::new_unchecked(gathered.elements, gathered.offsets, validity) }
                .into_array(),
        )
    })
}

struct GatheredList {
    elements: ArrayRef,
    offsets: ArrayRef,
}

struct ValidPieceGather<OutOffset> {
    new_offsets: BufferMut<OutOffset>,
    element_starts: BufferMut<u64>,
    element_lengths: BufferMut<u64>,
    output_elements: usize,
}

fn piecewise_list_elements_len_constant<S, Offset>(
    offsets: &[Offset],
    starts: &[S],
    length: usize,
) -> VortexResult<usize>
where
    S: UnsignedPType,
    Offset: UnsignedPType,
{
    if length == 0 {
        return Ok(0);
    }

    let mut total = 0usize;
    for start in starts {
        let start: usize = start.as_();
        let offset_range = &offsets[start..][..=length];
        let element_start: usize = offset_range[0].as_();
        let element_end: usize = offset_range[length].as_();
        total = total
            .checked_add(element_end - element_start)
            .ok_or_else(|| vortex_err!("List take output elements length overflow"))?;
    }
    Ok(total)
}

fn piecewise_list_elements_len_constant_validity<S, Offset>(
    offsets: &[Offset],
    starts: &[S],
    length: usize,
    data_validity: &Mask,
) -> VortexResult<usize>
where
    S: UnsignedPType,
    Offset: UnsignedPType,
{
    if length == 0 {
        return Ok(0);
    }

    let mut total = 0usize;
    for start in starts {
        let start: usize = start.as_();
        let additional = valid_piece_elements_len(offsets, data_validity, start, length)?;
        total = total
            .checked_add(additional)
            .ok_or_else(|| vortex_err!("List take output elements length overflow"))?;
    }
    Ok(total)
}

fn piecewise_list_elements_len<S, L, Offset>(
    offsets: &[Offset],
    starts: &[S],
    lengths: &[L],
) -> VortexResult<usize>
where
    S: UnsignedPType,
    L: UnsignedPType,
    Offset: UnsignedPType,
{
    let mut total = 0usize;
    for (&start, &length) in starts.iter().zip_eq(lengths) {
        let start: usize = start.as_();
        let length: usize = length.as_();
        let offset_range = &offsets[start..][..=length];
        let element_start: usize = offset_range[0].as_();
        let element_end: usize = offset_range[length].as_();
        total = total
            .checked_add(element_end - element_start)
            .ok_or_else(|| vortex_err!("List take output elements length overflow"))?;
    }
    Ok(total)
}

fn piecewise_list_elements_len_validity<S, L, Offset>(
    offsets: &[Offset],
    starts: &[S],
    lengths: &[L],
    data_validity: &Mask,
) -> VortexResult<usize>
where
    S: UnsignedPType,
    L: UnsignedPType,
    Offset: UnsignedPType,
{
    let mut total = 0usize;
    for (&start, &length) in starts.iter().zip_eq(lengths) {
        let start: usize = start.as_();
        let length: usize = length.as_();
        let additional = valid_piece_elements_len(offsets, data_validity, start, length)?;
        total = total
            .checked_add(additional)
            .ok_or_else(|| vortex_err!("List take output elements length overflow"))?;
    }
    Ok(total)
}

fn valid_piece_elements_len<Offset>(
    offsets: &[Offset],
    data_validity: &Mask,
    start: usize,
    length: usize,
) -> VortexResult<usize>
where
    Offset: UnsignedPType,
{
    let offset_range = &offsets[start..][..=length];
    let mut total = 0usize;
    for (data_idx, window) in (start..).zip(offset_range.windows(2)) {
        if !data_validity.value(data_idx) {
            continue;
        }
        let element_start: usize = window[0].as_();
        let element_end: usize = window[1].as_();
        total = total
            .checked_add(element_end - element_start)
            .ok_or_else(|| vortex_err!("List take output elements length overflow"))?;
    }
    Ok(total)
}

fn gather_piecewise_list_constant_length<S, Offset, OutOffset>(
    elements: &ArrayRef,
    offsets: &[Offset],
    starts: &[S],
    length: usize,
    output_len: usize,
    total_elements: usize,
) -> VortexResult<GatheredList>
where
    S: UnsignedPType,
    Offset: UnsignedPType,
    OutOffset: IntegerPType,
{
    let offsets_capacity = output_len
        .checked_add(1)
        .ok_or_else(|| vortex_err!("List take offsets length overflow"))?;
    let mut new_offsets = BufferMut::<OutOffset>::with_capacity(offsets_capacity);
    let mut element_starts = BufferMut::<u64>::with_capacity(starts.len());
    let mut element_lengths = BufferMut::<u64>::with_capacity(starts.len());
    let mut output_elements = 0usize;

    new_offsets.push(OutOffset::zero());
    for start in starts {
        let start: usize = start.as_();
        if length == 0 {
            continue;
        }

        let offset_range = &offsets[start..][..=length];
        let element_start: usize = offset_range[0].as_();
        let element_end: usize = offset_range[length].as_();
        for &offset in &offset_range[1..] {
            let offset: usize = offset.as_();
            let relative = offset - element_start;
            let output_offset = output_elements + relative;
            new_offsets.push(new_offset_value::<OutOffset>(output_offset));
        }

        let element_length = element_end - element_start;
        element_starts.push(element_start as u64);
        element_lengths.push(element_length as u64);
        output_elements += element_length;
    }
    debug_assert_eq!(output_elements, total_elements);

    let offsets = PrimitiveArray::new(new_offsets.freeze(), Validity::NonNullable).into_array();
    let multipliers = ConstantArray::new(1u64, element_starts.len()).into_array();
    // SAFETY: element ranges are derived from validated source list offsets, and total_elements is
    // the sum of the gathered element range lengths. Multiplier 1 preserves contiguous ranges.
    let element_indices = unsafe {
        PiecewiseSequenceArray::new_unchecked(
            element_starts.into_array(),
            element_lengths.into_array(),
            multipliers,
            total_elements,
        )
    };
    let elements = elements.take(element_indices.into_array())?;

    Ok(GatheredList { elements, offsets })
}

fn gather_piecewise_list_constant_length_validity<S, Offset, OutOffset>(
    elements: &ArrayRef,
    offsets: &[Offset],
    starts: &[S],
    length: usize,
    output_len: usize,
    total_elements: usize,
    data_validity: &Mask,
) -> VortexResult<GatheredList>
where
    S: UnsignedPType,
    Offset: UnsignedPType,
    OutOffset: IntegerPType,
{
    let offsets_capacity = output_len
        .checked_add(1)
        .ok_or_else(|| vortex_err!("List take offsets length overflow"))?;
    let mut gather = ValidPieceGather {
        new_offsets: BufferMut::<OutOffset>::with_capacity(offsets_capacity),
        element_starts: BufferMut::<u64>::with_capacity(output_len),
        element_lengths: BufferMut::<u64>::with_capacity(output_len),
        output_elements: 0,
    };

    gather.new_offsets.push(OutOffset::zero());
    for start in starts {
        let start: usize = start.as_();
        if length == 0 {
            continue;
        }

        gather_valid_piece(offsets, data_validity, start, length, &mut gather);
    }
    debug_assert_eq!(gather.output_elements, total_elements);

    let offsets =
        PrimitiveArray::new(gather.new_offsets.freeze(), Validity::NonNullable).into_array();
    let multipliers = ConstantArray::new(1u64, gather.element_starts.len()).into_array();
    // SAFETY: element ranges come only from valid source list rows. Source list construction
    // validated those row offsets, and null source rows produce no element range.
    let element_indices = unsafe {
        PiecewiseSequenceArray::new_unchecked(
            gather.element_starts.into_array(),
            gather.element_lengths.into_array(),
            multipliers,
            total_elements,
        )
    };
    let elements = elements.take(element_indices.into_array())?;

    Ok(GatheredList { elements, offsets })
}

fn gather_piecewise_list<S, L, Offset, OutOffset>(
    elements: &ArrayRef,
    offsets: &[Offset],
    starts: &[S],
    lengths: &[L],
    output_len: usize,
    total_elements: usize,
) -> VortexResult<GatheredList>
where
    S: UnsignedPType,
    L: UnsignedPType,
    Offset: UnsignedPType,
    OutOffset: IntegerPType,
{
    let offsets_capacity = output_len
        .checked_add(1)
        .ok_or_else(|| vortex_err!("List take offsets length overflow"))?;
    let mut new_offsets = BufferMut::<OutOffset>::with_capacity(offsets_capacity);
    let mut element_starts = BufferMut::<u64>::with_capacity(starts.len());
    let mut element_lengths = BufferMut::<u64>::with_capacity(lengths.len());
    let mut output_elements = 0usize;

    new_offsets.push(OutOffset::zero());
    for (&start, &length) in starts.iter().zip_eq(lengths) {
        let start: usize = start.as_();
        let length: usize = length.as_();
        if length == 0 {
            continue;
        }

        let offset_range = &offsets[start..][..=length];
        let element_start: usize = offset_range[0].as_();
        let element_end: usize = offset_range[length].as_();
        for &offset in &offset_range[1..] {
            let offset: usize = offset.as_();
            let relative = offset - element_start;
            let output_offset = output_elements + relative;
            new_offsets.push(new_offset_value::<OutOffset>(output_offset));
        }

        let element_length = element_end - element_start;
        element_starts.push(element_start as u64);
        element_lengths.push(element_length as u64);
        output_elements += element_length;
    }
    debug_assert_eq!(output_elements, total_elements);

    let offsets = PrimitiveArray::new(new_offsets.freeze(), Validity::NonNullable).into_array();
    let multipliers = ConstantArray::new(1u64, element_starts.len()).into_array();
    // SAFETY: element ranges are derived from validated source list offsets, and total_elements is
    // the sum of the gathered element range lengths. Multiplier 1 preserves contiguous ranges.
    let element_indices = unsafe {
        PiecewiseSequenceArray::new_unchecked(
            element_starts.into_array(),
            element_lengths.into_array(),
            multipliers,
            total_elements,
        )
    };
    let elements = elements.take(element_indices.into_array())?;

    Ok(GatheredList { elements, offsets })
}

fn gather_piecewise_list_validity<S, L, Offset, OutOffset>(
    elements: &ArrayRef,
    offsets: &[Offset],
    starts: &[S],
    lengths: &[L],
    output_len: usize,
    total_elements: usize,
    data_validity: &Mask,
) -> VortexResult<GatheredList>
where
    S: UnsignedPType,
    L: UnsignedPType,
    Offset: UnsignedPType,
    OutOffset: IntegerPType,
{
    let offsets_capacity = output_len
        .checked_add(1)
        .ok_or_else(|| vortex_err!("List take offsets length overflow"))?;
    let mut gather = ValidPieceGather {
        new_offsets: BufferMut::<OutOffset>::with_capacity(offsets_capacity),
        element_starts: BufferMut::<u64>::with_capacity(output_len),
        element_lengths: BufferMut::<u64>::with_capacity(output_len),
        output_elements: 0,
    };

    gather.new_offsets.push(OutOffset::zero());
    for (&start, &length) in starts.iter().zip_eq(lengths) {
        let start: usize = start.as_();
        let length: usize = length.as_();
        if length == 0 {
            continue;
        }

        gather_valid_piece(offsets, data_validity, start, length, &mut gather);
    }
    debug_assert_eq!(gather.output_elements, total_elements);

    let offsets =
        PrimitiveArray::new(gather.new_offsets.freeze(), Validity::NonNullable).into_array();
    let multipliers = ConstantArray::new(1u64, gather.element_starts.len()).into_array();
    // SAFETY: element ranges come only from valid source list rows. Source list construction
    // validated those row offsets, and null source rows produce no element range.
    let element_indices = unsafe {
        PiecewiseSequenceArray::new_unchecked(
            gather.element_starts.into_array(),
            gather.element_lengths.into_array(),
            multipliers,
            total_elements,
        )
    };
    let elements = elements.take(element_indices.into_array())?;

    Ok(GatheredList { elements, offsets })
}

fn gather_valid_piece<Offset, OutOffset>(
    offsets: &[Offset],
    data_validity: &Mask,
    start: usize,
    length: usize,
    gather: &mut ValidPieceGather<OutOffset>,
) where
    Offset: UnsignedPType,
    OutOffset: IntegerPType,
{
    let offset_range = &offsets[start..][..=length];
    for (data_idx, window) in (start..).zip(offset_range.windows(2)) {
        if !data_validity.value(data_idx) {
            gather
                .new_offsets
                .push(new_offset_value::<OutOffset>(gather.output_elements));
            continue;
        }

        let element_start: usize = window[0].as_();
        let element_end: usize = window[1].as_();
        let element_length = element_end - element_start;
        if element_length != 0 {
            gather.element_starts.push(element_start as u64);
            gather.element_lengths.push(element_length as u64);
            gather.output_elements += element_length;
        }
        gather
            .new_offsets
            .push(new_offset_value::<OutOffset>(gather.output_elements));
    }
}

fn new_offset_value<T: IntegerPType>(value: usize) -> T {
    T::from_usize(value).vortex_expect("output offset fits selected offset type")
}

#[cfg(test)]
mod test {
    use std::sync::Arc;

    use rstest::rstest;
    use vortex_buffer::buffer;
    use vortex_error::VortexResult;

    use crate::IntoArray as _;
    use crate::VortexSessionExecute;
    use crate::array_session;
    use crate::arrays::BoolArray;
    use crate::arrays::ConstantArray;
    use crate::arrays::ListArray;
    use crate::arrays::ListViewArray;
    use crate::arrays::PiecewiseSequenceArray;
    use crate::arrays::PrimitiveArray;
    use crate::arrays::listview::ListViewArrayExt;
    use crate::assert_arrays_eq;
    use crate::compute::conformance::take::test_take_conformance;
    use crate::dtype::DType;
    use crate::dtype::Nullability;
    use crate::dtype::PType::I32;
    use crate::scalar::Scalar;
    use crate::validity::Validity;

    #[test]
    fn nullable_take() {
        let mut ctx = array_session().create_execution_ctx();
        let list = ListArray::try_new(
            buffer![0i32, 5, 3, 4].into_array(),
            buffer![0, 2, 3, 4, 4].into_array(),
            Validity::Array(BoolArray::from_iter(vec![true, true, false, true]).into_array()),
            &mut ctx,
        )
        .unwrap()
        .into_array();

        let idx =
            PrimitiveArray::from_option_iter(vec![Some(0), None, Some(1), Some(3)]).into_array();

        let result = list.take(idx).unwrap();

        assert_eq!(
            result.dtype(),
            &DType::List(
                Arc::new(DType::Primitive(I32, Nullability::NonNullable)),
                Nullability::Nullable
            )
        );

        let result = result.execute::<ListViewArray>(&mut ctx).unwrap();

        assert_eq!(result.len(), 4);

        let element_dtype: Arc<DType> = Arc::new(I32.into());

        assert!(
            result
                .is_valid(0, &mut array_session().create_execution_ctx())
                .unwrap()
        );
        assert_eq!(
            result
                .execute_scalar(0, &mut array_session().create_execution_ctx())
                .unwrap(),
            Scalar::list(
                Arc::clone(&element_dtype),
                vec![0i32.into(), 5.into()],
                Nullability::Nullable
            )
        );

        assert!(
            result
                .is_invalid(1, &mut array_session().create_execution_ctx())
                .unwrap()
        );

        assert!(
            result
                .is_valid(2, &mut array_session().create_execution_ctx())
                .unwrap()
        );
        assert_eq!(
            result
                .execute_scalar(2, &mut array_session().create_execution_ctx())
                .unwrap(),
            Scalar::list(
                Arc::clone(&element_dtype),
                vec![3i32.into()],
                Nullability::Nullable
            )
        );

        assert!(
            result
                .is_valid(3, &mut array_session().create_execution_ctx())
                .unwrap()
        );
        assert_eq!(
            result
                .execute_scalar(3, &mut array_session().create_execution_ctx())
                .unwrap(),
            Scalar::list(element_dtype, vec![], Nullability::Nullable)
        );
    }

    #[test]
    fn null_index_ignores_out_of_bounds_payload() {
        let mut ctx = array_session().create_execution_ctx();
        let list = ListArray::try_new(
            buffer![1i32, 2, 3, 4].into_array(),
            buffer![0u32, 2, 4].into_array(),
            Validity::NonNullable,
            &mut ctx,
        )
        .unwrap()
        .into_array();

        let idx = PrimitiveArray::new(
            buffer![1u32, 99, 0],
            Validity::from_iter([true, false, true]),
        )
        .into_array();
        let result = list.take(idx).unwrap();

        let expected = ListArray::new(
            buffer![3i32, 4, 1, 2].into_array(),
            buffer![0u32, 2, 2, 4].into_array(),
            Validity::from_iter([true, false, true]),
            &mut ctx,
        );
        assert_arrays_eq!(result, expected, &mut ctx);
    }

    #[test]
    fn null_source_row_uses_valid_empty_output_range() {
        let mut ctx = array_session().create_execution_ctx();
        let list = ListArray::new(
            buffer![1i32, 2, 7, 8].into_array(),
            buffer![0u32, 2, 4].into_array(),
            Validity::from_iter([true, false]),
            &mut ctx,
        )
        .into_array();

        let idx = buffer![0u32, 1].into_array();
        let result = list.take(idx).unwrap();

        let expected = ListArray::new(
            buffer![1i32, 2].into_array(),
            buffer![0u32, 2, 2].into_array(),
            Validity::from_iter([true, false]),
            &mut ctx,
        );
        assert_arrays_eq!(result, expected, &mut ctx);
    }

    #[test]
    fn change_validity() {
        let mut ctx = array_session().create_execution_ctx();
        let list = ListArray::try_new(
            buffer![0i32, 5, 3, 4].into_array(),
            buffer![0, 2, 3].into_array(),
            Validity::NonNullable,
            &mut ctx,
        )
        .unwrap()
        .into_array();

        let idx = PrimitiveArray::from_option_iter(vec![Some(0), Some(1), None]).into_array();
        // since idx is nullable, the final list will also be nullable

        let result = list.take(idx).unwrap();
        assert_eq!(
            result.dtype(),
            &DType::List(
                Arc::new(DType::Primitive(I32, Nullability::NonNullable)),
                Nullability::Nullable
            )
        );
        let expected = ListArray::new(
            buffer![0i32, 5, 3].into_array(),
            buffer![0u32, 2, 3, 3].into_array(),
            Validity::from_iter([true, true, false]),
            &mut ctx,
        );
        assert_arrays_eq!(
            result,
            expected,
            &mut array_session().create_execution_ctx()
        );
    }

    #[test]
    fn non_nullable_take() {
        let mut ctx = array_session().create_execution_ctx();
        let list = ListArray::try_new(
            buffer![0i32, 5, 3, 4].into_array(),
            buffer![0, 2, 3, 3, 4].into_array(),
            Validity::NonNullable,
            &mut ctx,
        )
        .unwrap()
        .into_array();

        let idx = buffer![1, 0, 2].into_array();

        let result = list.take(idx).unwrap();

        assert_eq!(
            result.dtype(),
            &DType::List(
                Arc::new(DType::Primitive(I32, Nullability::NonNullable)),
                Nullability::NonNullable
            )
        );

        let result = result.execute::<ListViewArray>(&mut ctx).unwrap();

        assert_eq!(result.len(), 3);

        let element_dtype: Arc<DType> = Arc::new(I32.into());

        assert!(
            result
                .is_valid(0, &mut array_session().create_execution_ctx())
                .unwrap()
        );
        assert_eq!(
            result
                .execute_scalar(0, &mut array_session().create_execution_ctx())
                .unwrap(),
            Scalar::list(
                Arc::clone(&element_dtype),
                vec![3i32.into()],
                Nullability::NonNullable
            )
        );

        assert!(
            result
                .is_valid(1, &mut array_session().create_execution_ctx())
                .unwrap()
        );
        assert_eq!(
            result
                .execute_scalar(1, &mut array_session().create_execution_ctx())
                .unwrap(),
            Scalar::list(
                Arc::clone(&element_dtype),
                vec![0i32.into(), 5.into()],
                Nullability::NonNullable
            )
        );

        assert!(
            result
                .is_valid(2, &mut array_session().create_execution_ctx())
                .unwrap()
        );
        assert_eq!(
            result
                .execute_scalar(2, &mut array_session().create_execution_ctx())
                .unwrap(),
            Scalar::list(element_dtype, vec![], Nullability::NonNullable)
        );
    }

    #[test]
    fn piecewise_sequence_take() {
        let mut ctx = array_session().create_execution_ctx();
        let list = ListArray::try_new(
            buffer![0i32, 1, 2, 3, 4, 5, 6].into_array(),
            buffer![0u32, 2, 5, 5, 7].into_array(),
            Validity::NonNullable,
            &mut ctx,
        )
        .unwrap()
        .into_array();
        let idx = PiecewiseSequenceArray::try_new(
            buffer![1u64, 0].into_array(),
            buffer![2u64, 1].into_array(),
            ConstantArray::new(1u64, 2).into_array(),
            3,
        )
        .unwrap()
        .into_array();

        let result = list
            .take(idx)
            .unwrap()
            .execute::<ListViewArray>(&mut ctx)
            .unwrap();

        let element_dtype: Arc<DType> = Arc::new(I32.into());
        assert_eq!(
            result.execute_scalar(0, &mut ctx).unwrap(),
            Scalar::list(
                Arc::clone(&element_dtype),
                vec![2i32.into(), 3.into(), 4.into()],
                Nullability::NonNullable
            )
        );
        assert_eq!(
            result.execute_scalar(1, &mut ctx).unwrap(),
            Scalar::list(Arc::clone(&element_dtype), vec![], Nullability::NonNullable)
        );
        assert_eq!(
            result.execute_scalar(2, &mut ctx).unwrap(),
            Scalar::list(
                element_dtype,
                vec![0i32.into(), 1.into()],
                Nullability::NonNullable
            )
        );
    }

    #[test]
    fn piecewise_sequence_take_nullable_list_constant_lengths() -> VortexResult<()> {
        let mut ctx = array_session().create_execution_ctx();
        let list = ListArray::try_new(
            buffer![0i32, 1, 99, 100, 2, 3, 4, 5].into_array(),
            buffer![0u32, 2, 4, 7, 8].into_array(),
            Validity::Array(BoolArray::from_iter([true, false, true, true]).into_array()),
            &mut ctx,
        )?
        .into_array();
        let idx = PiecewiseSequenceArray::try_new(
            buffer![0u64].into_array(),
            ConstantArray::new(4u64, 1).into_array(),
            ConstantArray::new(1u64, 1).into_array(),
            4,
        )?
        .into_array();

        let result = list.take(idx)?.execute::<ListViewArray>(&mut ctx)?;
        assert_eq!(result.offset_at(0, &mut ctx), 0);
        assert_eq!(result.size_at(0, &mut ctx), 2);
        assert_eq!(result.offset_at(1, &mut ctx), 2);
        assert_eq!(result.size_at(1, &mut ctx), 0);
        assert_eq!(result.offset_at(2, &mut ctx), 2);
        assert_eq!(result.size_at(2, &mut ctx), 3);
        assert_eq!(result.offset_at(3, &mut ctx), 5);
        assert_eq!(result.size_at(3, &mut ctx), 1);

        let element_dtype: Arc<DType> = Arc::new(I32.into());
        assert_eq!(
            result.execute_scalar(0, &mut ctx)?,
            Scalar::list(
                Arc::clone(&element_dtype),
                vec![0i32.into(), 1.into()],
                Nullability::Nullable
            )
        );
        assert!(result.is_invalid(1, &mut ctx)?);
        assert_eq!(
            result.execute_scalar(2, &mut ctx)?,
            Scalar::list(
                Arc::clone(&element_dtype),
                vec![2i32.into(), 3.into(), 4.into()],
                Nullability::Nullable
            )
        );
        assert_eq!(
            result.execute_scalar(3, &mut ctx)?,
            Scalar::list(element_dtype, vec![5i32.into()], Nullability::Nullable)
        );
        Ok(())
    }

    #[test]
    fn piecewise_sequence_take_nullable_list_array_lengths() -> VortexResult<()> {
        let mut ctx = array_session().create_execution_ctx();
        let list = ListArray::try_new(
            buffer![0i32, 1, 99, 100, 2, 3, 4, 5].into_array(),
            buffer![0u32, 2, 4, 7, 8].into_array(),
            Validity::Array(BoolArray::from_iter([true, false, true, true]).into_array()),
            &mut ctx,
        )?
        .into_array();
        let idx = PiecewiseSequenceArray::try_new(
            buffer![1u64, 0].into_array(),
            buffer![2u64, 1].into_array(),
            ConstantArray::new(1u64, 2).into_array(),
            3,
        )?
        .into_array();

        let result = list.take(idx)?.execute::<ListViewArray>(&mut ctx)?;
        assert_eq!(result.offset_at(0, &mut ctx), 0);
        assert_eq!(result.size_at(0, &mut ctx), 0);
        assert_eq!(result.offset_at(1, &mut ctx), 0);
        assert_eq!(result.size_at(1, &mut ctx), 3);
        assert_eq!(result.offset_at(2, &mut ctx), 3);
        assert_eq!(result.size_at(2, &mut ctx), 2);

        let element_dtype: Arc<DType> = Arc::new(I32.into());
        assert!(result.is_invalid(0, &mut ctx)?);
        assert_eq!(
            result.execute_scalar(1, &mut ctx)?,
            Scalar::list(
                Arc::clone(&element_dtype),
                vec![2i32.into(), 3.into(), 4.into()],
                Nullability::Nullable
            )
        );
        assert_eq!(
            result.execute_scalar(2, &mut ctx)?,
            Scalar::list(
                element_dtype,
                vec![0i32.into(), 1.into()],
                Nullability::Nullable
            )
        );
        Ok(())
    }

    #[test]
    fn test_take_empty_array() {
        let mut ctx = array_session().create_execution_ctx();
        let list = ListArray::try_new(
            buffer![0i32, 5, 3, 4].into_array(),
            buffer![0].into_array(),
            Validity::NonNullable,
            &mut ctx,
        )
        .unwrap()
        .into_array();

        let idx = PrimitiveArray::empty::<i32>(Nullability::Nullable).into_array();

        let result = list.take(idx).unwrap();
        assert_eq!(
            result.dtype(),
            &DType::List(
                Arc::new(DType::Primitive(I32, Nullability::NonNullable)),
                Nullability::Nullable
            )
        );
        assert_eq!(result.len(), 0,);
    }

    #[rstest]
    #[case(ListArray::try_new(
        buffer![0i32, 1, 2, 3, 4, 5].into_array(),
        buffer![0, 2, 3, 5, 5, 6].into_array(),
        Validity::NonNullable,
        &mut array_session().create_execution_ctx(),
    ).unwrap())]
    #[case(ListArray::try_new(
        buffer![10i32, 20, 30, 40, 50].into_array(),
        buffer![0, 2, 3, 4, 5].into_array(),
        Validity::Array(BoolArray::from_iter(vec![true, false, true, true]).into_array()),
        &mut array_session().create_execution_ctx(),
    ).unwrap())]
    #[case(ListArray::try_new(
        buffer![1i32, 2, 3].into_array(),
        buffer![0, 0, 2, 2, 3].into_array(), // First and third are empty
        Validity::NonNullable,
        &mut array_session().create_execution_ctx(),
    ).unwrap())]
    #[case(ListArray::try_new(
        buffer![42i32, 43].into_array(),
        buffer![0, 2].into_array(),
        Validity::NonNullable,
        &mut array_session().create_execution_ctx(),
    ).unwrap())]
    #[case({
        let elements = buffer![0i32..200].into_array();
        let mut offsets = vec![0u64];
        for i in 1..=50 {
            offsets.push(offsets[i - 1] + (i as u64 % 5)); // Variable length lists
        }
        ListArray::try_new(
            elements,
            PrimitiveArray::from_iter(offsets).into_array(),
            Validity::NonNullable,
            &mut array_session().create_execution_ctx(),
        ).unwrap()
    })]
    #[case(ListArray::try_new(
        PrimitiveArray::from_option_iter([Some(1i32), None, Some(3), Some(4), None]).into_array(),
        buffer![0, 2, 3, 5].into_array(),
        Validity::NonNullable,
        &mut array_session().create_execution_ctx(),
    ).unwrap())]
    fn test_take_list_conformance(#[case] list: ListArray) {
        test_take_conformance(
            &list.into_array(),
            &mut array_session().create_execution_ctx(),
        );
    }

    #[test]
    fn test_u64_offset_accumulation_non_nullable() {
        let mut ctx = array_session().create_execution_ctx();
        let elements = buffer![0i32; 200].into_array();
        let offsets = buffer![0u8, 200].into_array();
        let list = ListArray::try_new(elements, offsets, Validity::NonNullable, &mut ctx)
            .unwrap()
            .into_array();

        // Take the same large list twice - would overflow u8 but works with u64.
        let idx = buffer![0u8, 0].into_array();
        let result = list.take(idx).unwrap();

        assert_eq!(result.len(), 2);

        let result_view = result.execute::<ListViewArray>(&mut ctx).unwrap();
        assert_eq!(result_view.len(), 2);
        assert!(
            result_view
                .is_valid(0, &mut array_session().create_execution_ctx())
                .unwrap()
        );
        assert!(
            result_view
                .is_valid(1, &mut array_session().create_execution_ctx())
                .unwrap()
        );
        let expected = ListArray::new(
            buffer![0i32; 400].into_array(),
            buffer![0u32, 200, 400].into_array(),
            Validity::NonNullable,
            &mut ctx,
        );
        assert_arrays_eq!(result_view, expected, &mut ctx);
    }

    #[test]
    fn test_u64_offset_accumulation_nullable() {
        let mut ctx = array_session().create_execution_ctx();
        let elements = buffer![0i32; 150].into_array();
        let offsets = buffer![0u8, 150, 150].into_array();
        let validity = BoolArray::from_iter(vec![true, false]).into_array();
        let list = ListArray::try_new(elements, offsets, Validity::Array(validity), &mut ctx)
            .unwrap()
            .into_array();

        // Take the same large list twice - would overflow u8 but works with u64.
        let idx = PrimitiveArray::from_option_iter(vec![Some(0u8), None, Some(0u8)]).into_array();
        let result = list.take(idx).unwrap();

        assert_eq!(result.len(), 3);

        let result_view = result.execute::<ListViewArray>(&mut ctx).unwrap();
        assert_eq!(result_view.len(), 3);
        assert!(
            result_view
                .is_valid(0, &mut array_session().create_execution_ctx())
                .unwrap()
        );
        assert!(
            result_view
                .is_invalid(1, &mut array_session().create_execution_ctx())
                .unwrap()
        );
        assert!(
            result_view
                .is_valid(2, &mut array_session().create_execution_ctx())
                .unwrap()
        );
        let expected = ListArray::new(
            buffer![0i32; 300].into_array(),
            buffer![0u32, 150, 150, 300].into_array(),
            Validity::from_iter([true, false, true]),
            &mut ctx,
        );
        assert_arrays_eq!(result_view, expected, &mut ctx);
    }

    /// Regression test for validity length mismatch bug.
    ///
    /// When source array has `Validity::Array(...)` and indices are non-nullable,
    /// the result validity must have length equal to indices.len(), not source.len().
    #[test]
    fn test_take_validity_length_mismatch_regression() {
        let mut ctx = array_session().create_execution_ctx();
        // Source array with explicit validity array (length 2).
        let list = ListArray::try_new(
            buffer![1i32, 2, 3, 4].into_array(),
            buffer![0, 2, 4].into_array(),
            Validity::Array(BoolArray::from_iter(vec![true, true]).into_array()),
            &mut ctx,
        )
        .unwrap()
        .into_array();

        // Take more indices than source length (4 vs 2) with non-nullable indices.
        let idx = buffer![0u32, 1, 0, 1].into_array();

        // This should not panic - result should have length 4.
        let result = list.take(idx).unwrap();
        assert_eq!(result.len(), 4);
        let expected = ListArray::new(
            buffer![1i32, 2, 3, 4, 1, 2, 3, 4].into_array(),
            buffer![0u32, 2, 4, 6, 8].into_array(),
            Validity::AllValid,
            &mut ctx,
        );
        assert_arrays_eq!(
            result,
            expected,
            &mut array_session().create_execution_ctx()
        );
    }
}