gengliangwang commented on a change in pull request #32590:
URL: https://github.com/apache/spark/pull/32590#discussion_r636057267
##########
File path:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/DeduplicateRelations.scala
##########
@@ -64,64 +78,74 @@ object DeduplicateRelations extends Rule[LogicalPlan] {
* @param existingRelations the known unique relations for a LogicalPlan
* @param plan the LogicalPlan that requires the deduplication
* @return (the new LogicalPlan which already deduplicate all duplicated
relations (if any),
- * all relations of the new LogicalPlan )
+ * all relations of the new LogicalPlan, whether the plan has
changed or not)
Review comment:
nit: `the plan has changed or not` => `the plan is changed or not`
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
