[ARROW-8331] [C++] arrow-compute-filter-benchmark fails to compile

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Fix Version/s: 0.17.0
• Component/s: C++
• Labels: pull-request-available
• External issue URL: https://github.com/apache/arrow/issues/17117
• Assignee: Wes McKinney
• Reporter: Wes McKinney
• Created: 03/Apr/20 21:39
• Updated: 11/Jan/23 07:59
• Resolved: 03/Apr/20 23:36

Description

Are the benchmarks not being built in CI?

../src/arrow/compute/kernels/filter_benchmark.cc:45:18: error: no matching function for call to 'Filter'
    ABORT_NOT_OK(Filter(&ctx, Datum(array), Datum(filter), &out));
                 ^~~~~~
../src/arrow/testing/gtest_util.h:109:18: note: expanded from macro 'ABORT_NOT_OK'
    auto _res = (expr);                                             \
                 ^~~~
../src/arrow/compute/kernels/filter.h:65:8: note: candidate function not viable: requires 5 arguments, but 4 were provided
Status Filter(FunctionContext* ctx, const Datum& values, const Datum& filter,
       ^
../src/arrow/compute/kernels/filter_benchmark.cc:66:18: error: no matching function for call to 'Filter'
    ABORT_NOT_OK(Filter(&ctx, Datum(array), Datum(filter), &out));
                 ^~~~~~
../src/arrow/testing/gtest_util.h:109:18: note: expanded from macro 'ABORT_NOT_OK'
    auto _res = (expr);                                             \
                 ^~~~
../src/arrow/compute/kernels/filter.h:65:8: note: candidate function not viable: requires 5 arguments, but 4 were provided
Status Filter(FunctionContext* ctx, const Datum& values, const Datum& filter,
       ^
../src/arrow/compute/kernels/filter_benchmark.cc:90:18: error: no matching function for call to 'Filter'
    ABORT_NOT_OK(Filter(&ctx, Datum(array), Datum(filter), &out));
                 ^~~~~~
../src/arrow/testing/gtest_util.h:109:18: note: expanded from macro 'ABORT_NOT_OK'
    auto _res = (expr);                                             \
                 ^~~~
../src/arrow/compute/kernels/filter.h:65:8: note: candidate function not viable: requires 5 arguments, but 4 were provided
Status Filter(FunctionContext* ctx, const Datum& values, const Datum& filter,
       ^
3 errors generated.

Issue Links

• links to: GitHub Pull Request #6834