hvanhovell commented on code in PR #40997:
URL: https://github.com/apache/spark/pull/40997#discussion_r1244515286


##########
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/object.scala:
##########
@@ -719,3 +720,99 @@ case class CoGroup(
   override protected def withNewChildrenInternal(
       newLeft: LogicalPlan, newRight: LogicalPlan): CoGroup = copy(left = 
newLeft, right = newRight)
 }
+
+object JoinWith {
+  /**
+   * find the trivially true predicates and automatically resolves them to 
both sides.
+   */
+  private[sql] def resolveSelfJoinCondition(resolver: Resolver, plan: Join): 
Join = {
+    val cond = plan.condition.map {

Review Comment:
   If you need a reason why this should be pushed into the analyzer then this 
would be it :)



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