You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@arrow.apache.org by we...@apache.org on 2022/05/26 01:23:33 UTC

[arrow] branch master updated: ARROW-16659: [C++] Remove ambiguous constructor for VectorKernel

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 cf414db758 ARROW-16659: [C++] Remove ambiguous constructor for VectorKernel
cf414db758 is described below

commit cf414db758a58320250b0129ba1f8e8b139d2513
Author: Tobias Zagorni <to...@zagorni.eu>
AuthorDate: Wed May 25 15:23:25 2022 -1000

    ARROW-16659: [C++] Remove ambiguous constructor for VectorKernel
    
    Closes #13235 from zagto/vectorkernel-constructor
    
    Authored-by: Tobias Zagorni <to...@zagorni.eu>
    Signed-off-by: Weston Pace <we...@gmail.com>
---
 cpp/src/arrow/compute/kernel.h | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/cpp/src/arrow/compute/kernel.h b/cpp/src/arrow/compute/kernel.h
index 27fb831636..6b1f23e78d 100644
--- a/cpp/src/arrow/compute/kernel.h
+++ b/cpp/src/arrow/compute/kernel.h
@@ -604,9 +604,6 @@ using VectorFinalize = std::function<Status(KernelContext*, std::vector<Datum>*)
 struct VectorKernel : public ArrayKernel {
   VectorKernel() = default;
 
-  VectorKernel(std::shared_ptr<KernelSignature> sig, ArrayKernelExec exec)
-      : ArrayKernel(std::move(sig), std::move(exec)) {}
-
   VectorKernel(std::vector<InputType> in_types, OutputType out_type, ArrayKernelExec exec,
                KernelInit init = NULLPTR, VectorFinalize finalize = NULLPTR)
       : ArrayKernel(std::move(in_types), std::move(out_type), std::move(exec),