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)