[ARROW-7303] [C++] Refactor benchmarks to use new Result APIs

• Type: Improvement
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 0.15.1
• Fix Version/s: 0.16.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/23590
• Assignee: Ben Kietzman
• Reporter: Ben Kietzman
• Created: 03/Dec/19 19:35
• Updated: 11/Jan/23 07:52
• Resolved: 05/Dec/19 08:10

Description

When building benchmarks, I get the following error:

../src/arrow/csv/converter_benchmark.cc:83:64: error: too many arguments to function call, expected 2, have 3
    ABORT_NOT_OK(converter->Convert(parser, 0 /* col_index */, &result));

This was introduced by ARROW-7236. I guess the CI didn't catch it because we don't currently build benchmarks? apitrou kszucs

Issue Links

• links to: GitHub Pull Request #5948