Skip to content

Invalid inter-file ordering in eq_properties() #20508

@alamb

Description

@alamb

Describe the bug

TLDR: just because the files themselves are sorted doesn't mean the partition streams are sorted.

  • eq_properties() in FileScanConfig blindly trusted output_ordering (set from Parquet sorting_columns metadata) without verifying that files within a group are in the correct inter-file order
  • EnforceSorting then removed SortExec based on this unvalidated ordering, producing wrong results when filesystem order didn't match data order

To Reproduce

No response

Expected behavior

No response

Additional context

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't working

    Type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions