Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[SPARK-10279][MLlib][PySpark][Docs] Add @since annotation to pyspark.mllib.util #8689

Closed
wants to merge 2 commits into from

Conversation

yu-iskw
Copy link
Contributor

@yu-iskw yu-iskw commented Sep 10, 2015

No description provided.

@SparkQA
Copy link

SparkQA commented Sep 10, 2015

Test build #42250 has finished for PR 8689 at commit ca0acfa.

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

@noel-smith
Copy link
Contributor

These changes are already included in #8685. Can probably drop this PR and merge SPARK-10279 with SPARK-10278.

@mengxr - Does that make sense to you?

@yu-iskw
Copy link
Contributor Author

yu-iskw commented Sep 16, 2015

@noel-smith sorry for the mistake. Instead of closing this PR, I will drop util.py from #8685.

@SparkQA
Copy link

SparkQA commented Sep 16, 2015

Test build #42533 has finished for PR 8689 at commit b9fe3a8.

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

@noel-smith
Copy link
Contributor

That's works too, thanks for resolving - LGTM

@yu-iskw
Copy link
Contributor Author

yu-iskw commented Sep 17, 2015

@noel-smith thanks for the review.

@asfgit asfgit closed this in 4a0b56e Sep 17, 2015
@mengxr
Copy link
Contributor

mengxr commented Sep 17, 2015

Merged into master. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants