make compilers happy

apache · Dec 23, 2020 · 8f6e26e · 8f6e26e
1 parent 32cd9a4
commit 8f6e26e
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/cpp/src/arrow/compute/kernels/scalar_string.cc b/cpp/src/arrow/compute/kernels/scalar_string.cc
@@ -1320,7 +1320,6 @@ struct Join {
         *out = scalar_right_type;
       }  // do we want to support scalar[list[str]] with array[str] ?
     } else {
-      const ArrayData& input = *batch[0].array();
       const ListArrayType list(batch[0].array());
       ArrayData* output = out->mutable_array();
 
@@ -1367,7 +1366,7 @@ struct Join {
             KERNEL_RETURN_IF_ERROR(ctx, builder.AppendNull());
           } else {
             if (strings->length() > 0) {
-              builder.Append(strings->GetView(0));
+              KERNEL_RETURN_IF_ERROR(ctx, builder.Append(strings->GetView(0)));
               for (int64_t j = 1; j < strings->length(); j++) {
                 KERNEL_RETURN_IF_ERROR(ctx, builder.AppendCurrent(separator));
                 KERNEL_RETURN_IF_ERROR(ctx, builder.AppendCurrent(strings->GetView(j)));