Dandandan commented on a change in pull request #971:
URL: https://github.com/apache/arrow-datafusion/pull/971#discussion_r709426469
##########
File path: datafusion/src/logical_plan/expr.rs
##########
@@ -934,6 +934,19 @@ impl Not for Expr {
}
}
+impl std::fmt::Display for Expr {
+ fn fmt(&self, f: &mut std::fmt::Formatter) -> std::fmt::Result {
+ match self {
+ Expr::BinaryExpr {
+ ref left,
+ ref right,
+ ref op,
+ } => write!(f, "{} {} {}", left, op, right),
Review comment:
This is not used at the moment, as the implementation uses debug.
--
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]
