[ARROW-5311] [C++] Return more specific invalid Status in Take kernel

• Type: Improvement
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 0.14.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/21774
• Assignee: Ben Kietzman
• Reporter: Joris Van den Bossche
• Created: 13/May/19 15:10
• Updated: 11/Jan/23 07:39
• Resolved: 16/May/19 12:43

Description

Currently the Take kernel returns generic Invalid Status for certain cases, that could use more specific error:

• indices of wrong type (eg floats) -> TypeError instead of Invalid?
• out of bounds index -> new IndexError ?

From review in https://github.com/apache/arrow/pull/4281

cc bkietz

Issue Links

• is related to: ARROW-772 [C++] Implement take kernel functions
• links to: GitHub Pull Request #4317