Skip to content

[Variant]: Implement DataType::ListView/LargeListView support for cast_to_variant kernel #8236

@liamzwbao

Description

@liamzwbao

Part of #6736

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

We implemented cast_to_variant for some primitive types in

Now we need to complete out type support for other Arrow types.

Describe the solution you'd like

Add support for the types named in the title as well.

Describe alternatives you've considered

  1. Add the corresponding match arms in the cast_to_variant kernel:

  2. add tests (e.g.

    fn test_cast_to_variant_int8() {
    run_test(
    Arc::new(Int8Array::from(vec![
    Some(i8::MIN),
    None,
    Some(-1),
    Some(1),
    Some(i8::MAX),
    ])),
    vec![
    Some(Variant::Int8(i8::MIN)),
    None,
    Some(Variant::Int8(-1)),
    Some(Variant::Int8(1)),
    Some(Variant::Int8(i8::MAX)),
    ],
    )
    }
    )

Metadata

Metadata

Assignees

Labels

enhancementAny new improvement worthy of a entry in the changelog

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions