[ARROW-16116] [C++] Properly handle non-nullable fields in Parquet reading

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 9.0.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/31526
• Assignee: David Li
• Reporter: David Li
• Created: 04/Apr/22 21:19
• Updated: 11/Jan/23 11:41
• Resolved: 04/May/22 13:19

Description

ARROW-15961 found that the Parquet Arrow reader wasn't respecting the nullable aspect of fields, we need to ensure that if we reconstruct an array for a non-nullable field, that it has no validity bitmap. We need to also add tests for this case, they're implicitly tested in a few places, but we should explicitly test this for all supported types.

Issue Links

• relates to: ARROW-15961 [C++] Check nullability when validating fields on batches or struct arrays
• links to: GitHub Pull Request #12829