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.

Sign up for GitHub

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

Jump to bottom

Don't Panic on Invalid Parquet Statistics #3577

Closed
tustvold opened this issue Jan 20, 2023 · 0 comments · Fixed by #4642 or #6295
Closed

Don't Panic on Invalid Parquet Statistics #3577

tustvold opened this issue Jan 20, 2023 · 0 comments · Fixed by #4642 or #6295
Labels
enhancement Any new improvement worthy of a entry in the changelog

Comments

@tustvold
Copy link
Contributor

Is your feature request related to a problem or challenge? Please describe what you are trying to do.

Invalid statistics will currently result in the parquet reader panicking, we should instead return an error

Describe the solution you'd like

Describe alternatives you've considered

Additional context
@tustvold tustvold added the enhancement Any new improvement worthy of a entry in the changelog label Jan 20, 2023
@tustvold tustvold changed the title Don't Panic on Invalid Statistics Don't Panic on Invalid Parquet Statistics Jan 20, 2023
@tustvold tustvold mentioned this issue Jan 20, 2023
Merged
@jackwener jackwener mentioned this issue Aug 4, 2023
Merged
samuelcolvin added a commit to samuelcolvin/arrow-rs that referenced this issue Aug 23, 2024
@samuelcolvin
Err on try_from_le_slice, fix apache#3577
9734024
@samuelcolvin samuelcolvin mentioned this issue Aug 23, 2024
Merged
alamb pushed a commit that referenced this issue Aug 26, 2024
@samuelcolvin
Err on try_from_le_slice (#6295)
ee2f75a 
* Err on try_from_le_slice, fix #3577

* format and changes

* small cleanup

* fix clippy

* add bad metadata test

* run test only if feature is enabled

* add MRE test

* fmt
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement Any new improvement worthy of a entry in the changelog
Projects
None yet
Milestone
No milestone
1 participant
@tustvold