Skip to content

[Variant] Support BinaryArray/LargeBinaryArray in addition to BinaryViewArray for Variant #8387

Closed

Description

@alamb

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

Describe the solution you'd like
Currently VariantArray requires BinaryViewArray but by default the Parquet reader produces BinaryArray -- this means that without a workaround, it is not possible to read data from the ParquetReader without converting

According to the Arrow canonical extension type (see apache/arrow#47456) BinaryArray is valid (as is BinaryViewArray)

I added a workaround in

But this ticket tracks the work to actually support BinaryArray properly in VariantArray

Describe alternatives you've considered

Additional context

Activity

  1. alamb commented on Sep 22, 2025

    @alamb
    ContributorAuthor
  2. changed the title [-][Variant] Support BinaryArray in addition to `BinaryViewArray` for Variant[/-] [+][Variant] Support `BinaryArray`/`LargeBinaryArray` in addition to `BinaryViewArray` for Variant[/+] on Sep 22, 2025
  3. AdamGS commented on Mar 24, 2026

    @AdamGS
    Contributor

    take

  4. AdamGS commented on Mar 24, 2026

    @AdamGS
    Contributor

    I think there are two general approaches here:

    1. Make variant array generic over the underlying type, and wrap it up in something to hide the generics.
    2. Just use inner ArrayRefs, and validate at the edges

    I'm starting with the second approach as it seems much simpler, but I assume there will be some performance cost.

  5. added a commit that references this issue on Apr 15, 2026
    72b4576
  6. added a commit that references this issue on Jun 2, 2026
    2f83c76
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions