Skip to content

Implement DictionaryArray support in eq_dyn, neq_dyn, lt_dyn, lt_eq_dyn, gt_dyn, gt_eq_dyn #1201

@alamb

Description

@alamb

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
Rationale make it fast and easy (more ergonomic) to compare arrays, both for DataFusion and other users. See rationale in #843

Describe the solution you'd like
The idea would basically be to implement the same thing as @viirya did for timestamp types in #1095 but for DictionaryArrays

That is the following code should not assert and compare the decimal arrays element by element:

let arr1: DictionaryArray<Int32Type> = ...;
let arr2: DictionaryArray<Int32Type> = ...;
let result = eq_dyn(&arr1, &arr2);
assert!(result.is_ok())

Describe alternatives you've considered
For completeness, we could also add eq_dict, neq_dict, etc as well, though I personally think focusing on the dyn kernels would be the easiest thing.

Additional context

other context:
I am trying to move datafusion over to using the dyn kernels here: apache/datafusion#1475

Metadata

Metadata

Assignees

No one assigned

    Labels

    arrowChanges to the arrow crateenhancementAny 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