peter-toth commented on code in PR #10473:
URL: https://github.com/apache/datafusion/pull/10473#discussion_r1650116827
##########
datafusion/expr/src/expr.rs:
##########
@@ -1461,6 +1462,176 @@ impl Expr {
| Expr::Placeholder(..) => false,
}
}
+
+ /// This method hashes the direct content of an expression node without
recursing into
+ /// its children. This is useful because in `CommonSubexprEliminate` we
can build up
+ /// the deep hash of a node and its descendants during the bottom-up phase
of the
+ /// first traversal and so avoid computing the hash of the node and then
the hash of
+ /// its descendants separately.
+ ///
+ /// As it is pretty easy to forget changing this method when `Expr`
changes the
+ /// implementation doesn't use wildcard patterns (`..`, `_`) to catch
changes
+ /// compile time.
Review Comment:
Fixed in
https://github.com/apache/datafusion/pull/10473/commits/ee61224f1a948c38ac99ae53d9d426bb69da9548.
--
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: github-unsubscr...@datafusion.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org
