[arrow] branch master updated: MINOR: [C++] Add check for result of MakeExecNode (#13849)

[westonpace]

This is an automated email from the ASF dual-hosted git repository.

westonpace pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new b7c94e2f33 MINOR: [C++] Add check for result of MakeExecNode (#13849)
b7c94e2f33 is described below

commit b7c94e2f330b628ac3d3c92972ad691c0ec454d4
Author: octalene <octalene....@pm.me>
AuthorDate: Wed Aug 10 17:53:51 2022 -0700

    MINOR: [C++] Add check for result of MakeExecNode (#13849)
    
    ARROW-16894 added benchmarks for asof join node. This PR adds a call to 
`ASSERT_OK` to satisfy the warning that the `Result` returned by `MakeExecNode` 
was not checked.
    
    Authored-by: Aldrin M <octalene....@pm.me>
    Signed-off-by: Weston Pace <weston.p...@gmail.com>
---
 cpp/src/arrow/compute/exec/asof_join_benchmark.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/cpp/src/arrow/compute/exec/asof_join_benchmark.cc 
b/cpp/src/arrow/compute/exec/asof_join_benchmark.cc
index af471a5013..543a4ece57 100644
--- a/cpp/src/arrow/compute/exec/asof_join_benchmark.cc
+++ b/cpp/src/arrow/compute/exec/asof_join_benchmark.cc
@@ -89,7 +89,7 @@ static void TableJoinOverhead(benchmark::State& state,
     ASSERT_OK_AND_ASSIGN(arrow::compute::ExecNode * join_node,
                          MakeExecNode(factory_name, plan.get(), input_nodes, 
options));
     AsyncGenerator<util::optional<ExecBatch>> sink_gen;
-    MakeExecNode("sink", plan.get(), {join_node}, SinkNodeOptions{&sink_gen});
+    ASSERT_OK(MakeExecNode("sink", plan.get(), {join_node}, 
SinkNodeOptions{&sink_gen}));
     state.ResumeTiming();
     ASSERT_FINISHES_OK(StartAndCollect(plan.get(), sink_gen));
   }