Skip to content

implement eq_dyn, neq_dyn, lt_dyn, lt_eq_dyn, gt_dyn, gt_eq_dyn for timestamp types #1086

@alamb

Description

@alamb

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
We are tryng to use the new dynamic comparison kernels added in #858 / #843 by @jimexist
in DataFusion: apache/datafusion#1475

It turns out that Datafusion also supports operations on the timestamp / time arrays as well so we can't use the dynamic kernels (yet)

Describe the solution you'd like
Add support for the following types to eq_dyn, neq_dyn, lt_dyn, lt_eq_dyn, gt_dyn, gt_eq_dyn (there is one macro needed):

DataType::Timestamp(TimeUnit::Nanosecond, _) => {
DataType::Timestamp(TimeUnit::Microsecond, _) => {
DataType::Timestamp(TimeUnit::Millisecond, _) => {
DataType::Timestamp(TimeUnit::Second, _) => {
DataType::Date32 => {
DataType::Date64 => {

Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.

Additional context

Metadata

Metadata

Assignees

No one assigned

    Labels

    arrowChanges to the arrow crateenhancementAny new improvement worthy of a entry in the changeloggood first issueGood for newcomers

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions