lidavidm commented on a change in pull request #10557:
URL: https://github.com/apache/arrow/pull/10557#discussion_r662293004
##########
File path: cpp/src/arrow/compute/kernels/scalar_if_else_test.cc
##########
@@ -316,5 +318,165 @@ TEST_F(TestIfElseKernel, IfElseDispatchBest) {
CheckDispatchBest(name, {null(), uint8(), int8()}, {boolean(), int16(),
int16()});
}
+void CheckVarArgs(const std::string& name, const std::vector<Datum>& inputs,
Review comment:
It's also somewhat overlapping with CheckScalar now that I look at it.
I'll take a look and see if I can't consolidate all three. (I may do so in a
different PR until we decide what to do with the implementation here, if we
want to split it up into separate 'case' and 'when' functions or not as
suggested.)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
