[ARROW-13589] [C++] Reconcile ValidateArray and ValidateArrayFull

• Type: Task
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 7.0.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/29234
• Assignee: Antoine Pitrou
• Reporter: Antoine Pitrou
• Created: 09/Aug/21 17:22
• Updated: 11/Jan/23 08:34
• Resolved: 17/Nov/21 09:59

Description

Instead of having entirely separate implementations for these functions, it seems the expensive checks can simply be toggled using a boolean flag inside a common implementation (as is already done for Scalar::Validate). Whether or not this makes the code more readable / shorter / more maintainable must be evaluated.

Issue Links

• links to: GitHub Pull Request #11670