Github user mgaido91 commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22284#discussion_r214270567
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/planning/QueryPlanner.scala
 ---
    @@ -81,7 +81,7 @@ abstract class QueryPlanner[PhysicalPlan <: 
TreeNode[PhysicalPlan]] {
                   childPlans.map { childPlan =>
                     // Replace the placeholder by the child plan
                     candidateWithPlaceholders.transformUp {
    -                  case p if p == placeholder => childPlan
    +                  case p if p.eq(placeholder) => childPlan
    --- End diff --
    
    not sure what you mean exactly here, may you elaborate a bit more please? 
Thanks.


---

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

Reply via email to