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

fix(rust): unable to read delta table when table contains both null and non-null add stats #2476

Merged
merged 3 commits into from
May 6, 2024

Conversation

yjshen
Copy link
Contributor

@yjshen yjshen commented May 3, 2024

Description

To fix the issue when a delta table contains add action with stats_parsed: null.

As shown in the test case, 001.json contains an Add action with stats, while 002.json contains an Add action with stats_parsed: null, before this fix, it will complain:

Arrow { source: InvalidArgumentError("all columns in a record batch must have the same length") }

The issue is that the array for num_records has two values, while for other stats such as null_count, the None value is filtered out by flat_map, so there is only one value in the array.

Related Issue(s)

closes #2477

Documentation

@github-actions github-actions bot added the binding/rust Issues for the Rust crate label May 3, 2024
@ion-elgreco ion-elgreco enabled auto-merge (squash) May 6, 2024 21:13
@ion-elgreco ion-elgreco force-pushed the fix_stats_not_always_present branch from 527328f to cb7e0d9 Compare May 6, 2024 21:34
@ion-elgreco ion-elgreco merged commit e370d34 into delta-io:main May 6, 2024
20 checks passed
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

unable to read delta table when table contains both null and non-null add stats
2 participants