Skip to content

Relax sort fallback constraints #1854

@mbutrovich

Description

@mbutrovich

What is the problem the feature request solves?

I came across this TODO in the code base:

// TODO: Remove this constraint when we upgrade to new arrow-rs including

The associated PR (apache/arrow-rs#6225) merged and the issue (apache/arrow-rs#6226) is closed, and we should be on a new enough version of arrow-rs.

There was a Comet issue (#807) and associated PR (#811) to fall back in the problematic scenario.

Describe the potential solution

Relax the fallback constraints and modify/add tests.

Additional context

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions