Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22284#discussion_r214227012
--- 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 --
As the placeholders are collected from `candidateWithPlaceholders`, I think
we will definitely have a matched child plan by reference equality here, right?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
