jlfsdtc commented on code in PR #44897:
URL: https://github.com/apache/spark/pull/44897#discussion_r1467433167


##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Expression.scala:
##########
@@ -310,17 +310,13 @@ abstract class Expression extends TreeNode[Expression] {
   /**
    * Returns true when two expressions will always compute the same result, 
even if they differ
    * cosmetically (i.e. capitalization of names in attributes may be 
different).
-   *
-   * See [[Canonicalize]] for more details.

Review Comment:
   Maybe can`Expression.canonicalized` to replace this `Canonicalize`



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to