GitHub user gatorsmile opened a pull request:

    https://github.com/apache/spark/pull/21197

    [SPARK-23971] [BACKPORT-2.3] Should not leak Spark sessions across test 
suites

    This PR is to backport the PR https://github.com/apache/spark/pull/21058 to 
Apache 2.3. This should be the cause why we saw the test regressions in Apache 
2.3 branches: 
    
    
https://amplab.cs.berkeley.edu/jenkins/job/spark-branch-2.3-test-sbt-hadoop-2.6/317/testReport/org.apache.spark.sql.execution.datasources.parquet/ParquetQuerySuite/SPARK_15678__not_use_cache_on_overwrite/history/
    
    
https://amplab.cs.berkeley.edu/jenkins/job/spark-branch-2.3-test-sbt-hadoop-2.7/318/testReport/junit/org.apache.spark.sql/DataFrameSuite/inputFiles/history/
    
    --- 
    
    ## What changes were proposed in this pull request?
    
    Many suites currently leak Spark sessions (sometimes with stopped 
SparkContexts) via the thread-local active Spark session and default Spark 
session. We should attempt to clean these up and detect when this happens to 
improve the reproducibility of tests.
    
    ## How was this patch tested?
    
    Existing tests


You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/gatorsmile/spark backportSPARK-23971

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/spark/pull/21197.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #21197
    
----
commit c1c63776687eef0be8f2675b3c247f793b9ea05e
Author: Eric Liang <ekl@...>
Date:   2018-04-13T05:30:59Z

    [SPARK-23971] Should not leak Spark sessions across test suites
    
    ## What changes were proposed in this pull request?
    
    Many suites currently leak Spark sessions (sometimes with stopped 
SparkContexts) via the thread-local active Spark session and default Spark 
session. We should attempt to clean these up and detect when this happens to 
improve the reproducibility of tests.
    
    ## How was this patch tested?
    
    Existing tests
    
    Author: Eric Liang <[email protected]>
    
    Closes #21058 from ericl/clear-session.

----


---

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

Reply via email to