Skip to content

[SPARK-8511][pyspark] Modify a test to remove a saved model in regression.py#6926

Closed
yu-iskw wants to merge 2 commits into
apache:masterfrom
yu-iskw:SPARK-8511
Closed

[SPARK-8511][pyspark] Modify a test to remove a saved model in regression.py#6926
yu-iskw wants to merge 2 commits into
apache:masterfrom
yu-iskw:SPARK-8511

Conversation

@yu-iskw

@yu-iskw yu-iskw commented Jun 21, 2015

Copy link
Copy Markdown
Contributor

@SparkQA

SparkQA commented Jun 21, 2015

Copy link
Copy Markdown

Test build #35401 has finished for PR 6926 at commit 4a01c9e.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@yu-iskw

yu-iskw commented Jun 21, 2015

Copy link
Copy Markdown
Contributor Author

Jenkins, test this please.

@jkbradley

Copy link
Copy Markdown
Member

This actually appears in even more places in MLlib. Would you mind fixing those too with this JIRA/PR? Thanks for catching this issue!

@yu-iskw

yu-iskw commented Jun 21, 2015

Copy link
Copy Markdown
Contributor Author

@jkbradley sure! will do!

@SparkQA

SparkQA commented Jun 21, 2015

Copy link
Copy Markdown

Test build #35405 has finished for PR 6926 at commit 4a01c9e.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@SparkQA

SparkQA commented Jun 21, 2015

Copy link
Copy Markdown

Test build #35412 has finished for PR 6926 at commit 7cd0948.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.

@yu-iskw

yu-iskw commented Jun 22, 2015

Copy link
Copy Markdown
Contributor Author

Jenkins, test this please.

@jkbradley

Copy link
Copy Markdown
Member

LGTM pending tests
Thanks!

@yu-iskw

yu-iskw commented Jun 22, 2015

Copy link
Copy Markdown
Contributor Author

@jkbradley Thank you for reviewing it!

@SparkQA

SparkQA commented Jun 22, 2015

Copy link
Copy Markdown

Test build #35434 has finished for PR 6926 at commit 7cd0948.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class Module(object):

@AmplabJenkins

Copy link
Copy Markdown

Merged build finished. Test FAILed.

@jkbradley

Copy link
Copy Markdown
Member

The tests passed, but posting here failed. I'll merge with master and branch-1.4

@asfgit asfgit closed this in 5d89d9f Jun 22, 2015
asfgit pushed a commit that referenced this pull request Jun 22, 2015
…ession.py`

[[SPARK-8511] Modify a test to remove a saved model in `regression.py` - ASF JIRA](https://issues.apache.org/jira/browse/SPARK-8511)

Author: Yu ISHIKAWA <yuu.ishikawa@gmail.com>

Closes #6926 from yu-iskw/SPARK-8511 and squashes the following commits:

7cd0948 [Yu ISHIKAWA] Use `shutil.rmtree()` to temporary directories for saving model testings, instead of `os.removedirs()`
4a01c9e [Yu ISHIKAWA] [SPARK-8511][pyspark] Modify a test to remove a saved model in `regression.py`

(cherry picked from commit 5d89d9f)
Signed-off-by: Joseph K. Bradley <joseph@databricks.com>

Conflicts:
	python/pyspark/mllib/tests.py
@yu-iskw

yu-iskw commented Jun 22, 2015

Copy link
Copy Markdown
Contributor Author

@jkbradley thank you for merging it!

nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 22, 2015
…ession.py`

[[SPARK-8511] Modify a test to remove a saved model in `regression.py` - ASF JIRA](https://issues.apache.org/jira/browse/SPARK-8511)

Author: Yu ISHIKAWA <yuu.ishikawa@gmail.com>

Closes apache#6926 from yu-iskw/SPARK-8511 and squashes the following commits:

7cd0948 [Yu ISHIKAWA] Use `shutil.rmtree()` to temporary directories for saving model testings, instead of `os.removedirs()`
4a01c9e [Yu ISHIKAWA] [SPARK-8511][pyspark] Modify a test to remove a saved model in `regression.py`

(cherry picked from commit 5d89d9f)
Signed-off-by: Joseph K. Bradley <joseph@databricks.com>

Conflicts:
	python/pyspark/mllib/tests.py
animeshbaranawal pushed a commit to animeshbaranawal/spark that referenced this pull request Jun 25, 2015
…ession.py`

[[SPARK-8511] Modify a test to remove a saved model in `regression.py` - ASF JIRA](https://issues.apache.org/jira/browse/SPARK-8511)

Author: Yu ISHIKAWA <yuu.ishikawa@gmail.com>

Closes apache#6926 from yu-iskw/SPARK-8511 and squashes the following commits:

7cd0948 [Yu ISHIKAWA] Use `shutil.rmtree()` to temporary directories for saving model testings, instead of `os.removedirs()`
4a01c9e [Yu ISHIKAWA] [SPARK-8511][pyspark] Modify a test to remove a saved model in `regression.py`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants