[ARROW-9049] [C++] Add a Result<> returning method for for constructing a dictionary

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Duplicate
• Component/s: C++
• External issue URL: https://github.com/apache/arrow/issues/25164
• Assignee: Micah Kornfield
• Reporter: Micah Kornfield
• Created: 05/Jun/20 23:25
• Updated: 11/Jan/23 08:04
• Resolved: 23/Nov/21 17:07

Description

Dictionary types require a signed integer index type. Today there is a DCHECK that this is the case in the constructor.

When reading data from an unknown source it is possible due to corruption (or user error) that the dictionary index type is not signed. We should add a method that checks for signedness and use that at all system boundaries to validate input data.

Issue Links

• is fixed by: ARROW-7592 [C++] Fix crashes on corrupt IPC input