Weston Pace created ARROW-16811:
-----------------------------------
Summary: [C++] Remove default exec context from Expression::Bind
Key: ARROW-16811
URL: https://issues.apache.org/jira/browse/ARROW-16811
Project: Apache Arrow
Issue Type: Improvement
Components: C++
Reporter: Weston Pace
This came up in https://github.com/apache/arrow/pull/13355.
It is maybe not very intuitive that Expression::Bind would require an
ExecContext and so we never provided one. However, when binding expressions we
need to lookup kernels, and that requires a function registry. Defaulting to
default_exec_context is something that should be done at a higher level and so
we should not allow ExecContext to be omitted when calling Bind.
Furthermore, [~rtpsw] has suggested that we might want to split
Expression::Bind into two variants. One which requires an ExecContext and one
which does not (but fails if it encounters a "call").
--
This message was sent by Atlassian Jira
(v8.20.7#820007)
