cloud-fan commented on code in PR #38379:
URL: https://github.com/apache/spark/pull/38379#discussion_r1003905057
##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala:
##########
@@ -242,7 +242,13 @@ abstract class Expression extends TreeNode[Expression] {
* This means that the lazy `cannonicalized` is called and computed only on
the root of the
* adjacent expressions.
*/
- lazy val canonicalized: Expression = {
+ lazy val canonicalized: Expression = withCanonicalizedChildren
+
+ /**
+ * The default process of canonicalization. It is a one pass, bottum-up
expression tree
+ * computation based oncanonicalizing children before canonicalizing the
current node.
+ */
+ final protected def withCanonicalizedChildren: Expression = {
Review Comment:
just name it `doCanonicalize`?
--
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: reviews-unsubscr...@spark.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
