Mingliang Zhu created SPARK-46584:
-------------------------------------

             Summary: Remove invalid attachCleanupResourceChecker in JoinSuite
                 Key: SPARK-46584
                 URL: https://issues.apache.org/jira/browse/SPARK-46584
             Project: Spark
          Issue Type: Improvement
          Components: SQL
    Affects Versions: 3.5.0
            Reporter: Mingliang Zhu


AttachCleanupResourceChecker in JoinSuite is invalid:

1. The matching of SortExec needs to be in `QueryExecution.executePlan` not 
`QueryExecution.sparkPlan`, The correct way is 
`foreachUp(df.queryExecution.executedPlan)\{f()}`.
2. `Mockito` counts the number of function calls, only for objects after `spy`. 
Calls to the original object are not counted. Therefore, when using 
df.queryExecution.executedPlan correctly to match, count is always 0.
3. Not all SortMergeJoin joinTypes will trigger cleanupResources(), such as 
'full outer join'.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@spark.apache.org
For additional commands, e-mail: issues-h...@spark.apache.org

Reply via email to