bkietz commented on a change in pull request #10397:
URL: https://github.com/apache/arrow/pull/10397#discussion_r640897940
##########
File path: cpp/src/arrow/compute/exec/expression.h
##########
@@ -64,8 +64,11 @@ class ARROW_EXPORT Expression {
/// Bind this expression to the given input type, looking up Kernels and
field types.
/// Some expression simplification may be performed and implicit casts will
be inserted.
/// Any state necessary for execution will be initialized and returned.
- Result<Expression> Bind(ValueDescr in, compute::ExecContext* = NULLPTR)
const;
- Result<Expression> Bind(const Schema& in_schema, compute::ExecContext* =
NULLPTR) const;
+ Result<Expression> Bind(ValueDescr in, ExecContext* = NULLPTR) const;
+ Result<Expression> Bind(const Schema& in_schema, ExecContext* = NULLPTR)
const;
+
+ Result<Expression> BindFlattened(ValueDescr in, ExecContext* = NULLPTR)
const;
Review comment:
No, this should be removed.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
