[ARROW-6929] [C++] ValidateArray is out of sync with the ListArray IPC specification

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.16.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/23251
• Assignee: Ben Kietzman
• Reporter: Micah Kornfield
• Created: 18/Oct/19 04:08
• Updated: 11/Jan/23 07:50
• Resolved: 24/Jan/20 04:58

Description

• It appears to check that null values take zero space
• It still checks for a begin offset of 0 if the array isn't sliced (technically this doesn't seem necessary and it could be non-zero even if the array wasn't sliced.)
• I think it also fails if an array is sliced to truncate it since it should compare length to data_extent instead of last_offset.

Issue Links

• links to: GitHub Pull Request #6269