Skip to content

Conversation

@jonded94
Copy link
Contributor

@jonded94 jonded94 commented Feb 12, 2026

Which issue does this PR close?

Related to

Rationale for this change

In #9374, I only came up with a unit-test that didn't really throw the message "Not all children array length are the same!" error the issue #9370 is about. In this PR, tests are introduced that are able to reproduce this issue end-to-end and are expected to still panic. In test test_list_struct_page_boundary_desync_produces_length_mismatch, we exactly get the error message the original issue is about, while test_row_selection_list_column_v2_page_boundary_skip shows a slightly different error message.

What changes are included in this PR?

Two test are introduced, which currently are expected to panic:

  • test_row_selection_list_column_v2_page_boundary_skip
  • test_list_struct_page_boundary_desync_produces_length_mismatch

Are there any user-facing changes?

No

Copy link
Contributor

@alamb alamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @jonded94 - this looks 👍 to me

@alamb alamb merged commit 33c4032 into apache:main Feb 12, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

parquet Changes to the parquet crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants