[ARROW-16929] [C++] Remove ExecBatchIterator and usages thereof

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 9.0.0
• Component/s: C++
• External issue URL: https://github.com/apache/arrow/issues/32248
• Assignee: Wes McKinney
• Reporter: Wes McKinney
• Created: 28/Jun/22 21:48
• Updated: 11/Jan/23 12:47
• Resolved: 30/Jul/22 00:45

Description

The only place left using it is in GroupBy in arrow/compute/exec/aggregate.cc. This can be refactored to use ExecSpan.

As part of this removal, we should adapt the benchmarks for ExecSpanIterator to demonstrate the performance improvement there

Issue Links

• is a child of: ARROW-16755 [C++] Improve array expression and kernel evaluation performance on small inputs