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

Add tests for outer nullability of structs in Parquet #17595

Closed
coastalwhite opened this issue Jul 12, 2024 · 0 comments · Fixed by #17941
Closed

Add tests for outer nullability of structs in Parquet #17595

coastalwhite opened this issue Jul 12, 2024 · 0 comments · Fixed by #17941
Assignees
@coastalwhite
Labels
A-io-parquet Area: reading/writing Parquet files accepted Ready for implementation chore Maintenance work that does not impact the user

Comments

@coastalwhite
Copy link
Collaborator

We should add tests for outer nullability of structs for the Parquet reader
@coastalwhite coastalwhite added the chore Maintenance work that does not impact the user label Jul 12, 2024
@coastalwhite coastalwhite self-assigned this Jul 12, 2024
@coastalwhite coastalwhite added the A-io-parquet Area: reading/writing Parquet files label Jul 12, 2024
@coastalwhite coastalwhite linked a pull request Jul 31, 2024 that will close this issue
fix: Several parquet reader/writer regressions #17941
Merged
@c-peters c-peters added the accepted Ready for implementation label Aug 5, 2024
@c-peters c-peters added this to Backlog Aug 5, 2024
@c-peters c-peters moved this to Done in Backlog Aug 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@coastalwhite coastalwhite

Labels
A-io-parquet Area: reading/writing Parquet files accepted Ready for implementation chore Maintenance work that does not impact the user
Projects
Backlog
Archived in project
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: Several parquet reader/writer regressions coastalwhite/polars
2 participants
@coastalwhite @c-peters