Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Previously valid queries now fail with "does not satisfy parent distribution requirements: SinglePartition" #2831

Open
adamfaulkner-at opened this issue Aug 26, 2024 · 1 comment
Assignees
Labels
binding/rust Issues for the Rust crate bug Something isn't working
Milestone

Comments

@adamfaulkner-at
Copy link
Contributor

Environment

Delta-rs version: 0.19.0 (also tested against main)

Binding: Rust

Environment: MacOS, linux

  • Cloud provider: AWS
  • OS: MacOS, Linux
  • Other:

Bug

What happened:

After performing the following operations:

  1. Create a table
  2. Perform two writes to the table
  3. Re-open the table.
  4. Perform a query that filters all data, and sorts on the column used for filtering

The query fails with this error:

called `Result::unwrap()` on an `Err` value: Context("SanityCheckPlan", Plan("Child: [\"CoalesceBatchesExec: target_batch_size=8192\", \"  FilterExec: a@0 > s\", \"    DeltaScan\", \"      ParquetExec: file_groups={0 groups: []}, projection=[a], predicate=a@0 > s, pruning_predicate=CASE WHEN a_null_count@1 = a_row_count@2 THEN false ELSE a_max@0 > s END, required_guarantees=[]\"] does not satisfy parent distribution requirements: SinglePartition"))

What you expected to happen:

I expected this query to continue to work like it did prior to 0.19.0.

How to reproduce it:

I've added a new failing test in this PR: #2830

More details:

I suspect this is somehow related to predicate pushdown excluding all parquet files from the DeltaScan

@adamfaulkner-at
Copy link
Contributor Author

Hello! I actually have a fix now:
#2830

@rtyler rtyler self-assigned this Oct 8, 2024
@rtyler rtyler added the binding/rust Issues for the Rust crate label Oct 8, 2024
@rtyler rtyler added this to the Rust v1.0.0 milestone Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
binding/rust Issues for the Rust crate bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants
@rtyler @adamfaulkner-at and others