GitHub user jkbradley opened a pull request:

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

    [SPARK-11869] [ML] Clean up TempDirectory properly in ML tests

    Need to remove parent directory (```className```) rather than just tempDir 
(```className/random_name```)
    
    I tested this with IDFSuite, which has 2 read/write tests, and it fixes the 
problem.
    
    CC: @mengxr  Can you confirm this is fine?  I believe it is since the same 
```random_name``` is used for all tests in a suite; we basically have an extra 
unneeded level of nesting.

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

    $ git pull https://github.com/jkbradley/spark tempdir-cleanup

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

    https://github.com/apache/spark/pull/9851.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 #9851
    
----
commit 3dec507913ede08c1e1d127feea654f713ce1b17
Author: Joseph K. Bradley <[email protected]>
Date:   2015-11-20T00:06:18Z

    clean up TempDirectory properly in ML tests

----


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---

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

Reply via email to