Skip to content

[MLlib] Remove transform(dataset: RDD[String]) from Word2Vec public API #2010

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

Closed
wants to merge 1 commit into from

Conversation

Ishiihara
Copy link
Contributor

@mengxr
Remove transform(dataset: RDD[String]) from public API.

@mengxr
Copy link
Contributor

mengxr commented Aug 18, 2014

Jenkins, test this please.

1 similar comment
@mengxr
Copy link
Contributor

mengxr commented Aug 18, 2014

Jenkins, test this please.

@SparkQA
Copy link

SparkQA commented Aug 18, 2014

QA tests have started for PR 2010 at commit 17b1031.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Aug 18, 2014

QA tests have finished for PR 2010 at commit 17b1031.

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

@mengxr
Copy link
Contributor

mengxr commented Aug 18, 2014

LGTM. Merged into master and branch-1.1. Thanks!

asfgit pushed a commit that referenced this pull request Aug 18, 2014
mengxr
Remove  transform(dataset: RDD[String]) from public API.

Author: Liquan Pei <liquanpei@gmail.com>

Closes #2010 from Ishiihara/Word2Vec-api and squashes the following commits:

17b1031 [Liquan Pei] remove transform(dataset: RDD[String]) from public API

(cherry picked from commit 9306b8c)
Signed-off-by: Xiangrui Meng <meng@databricks.com>
@asfgit asfgit closed this in 9306b8c Aug 18, 2014
xiliu82 pushed a commit to xiliu82/spark that referenced this pull request Sep 4, 2014
mengxr
Remove  transform(dataset: RDD[String]) from public API.

Author: Liquan Pei <liquanpei@gmail.com>

Closes apache#2010 from Ishiihara/Word2Vec-api and squashes the following commits:

17b1031 [Liquan Pei] remove transform(dataset: RDD[String]) from public API
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.

3 participants