-
Notifications
You must be signed in to change notification settings - Fork 28.5k
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
[MLLIB] minor update to word2vec #2043
Conversation
Looks good to me. |
@Ishiihara I added more changes. We don't need |
QA tests have started for PR 2043 at commit
|
QA tests have finished for PR 2043 at commit
|
Jenkins, retest this please. |
The Jenkins working folder is not clean. |
QA tests have started for PR 2043 at commit
|
QA tests have finished for PR 2043 at commit
|
very minor update Ishiihara Author: Xiangrui Meng <meng@databricks.com> Closes #2043 from mengxr/minor-w2v and squashes the following commits: be649fd [Xiangrui Meng] remove map because we only need append eccefcc [Xiangrui Meng] minor updates to word2vec (cherry picked from commit 1870dba) Signed-off-by: Xiangrui Meng <meng@databricks.com>
I've merged this into master and branch-1.1. |
Sorry I didn't realize this in #2043. Ishiihara Author: Xiangrui Meng <meng@databricks.com> Closes #2049 from mengxr/more-w2v and squashes the following commits: 050b1c5 [Xiangrui Meng] output shuffle data directly (cherry picked from commit 0a984aa) Signed-off-by: Xiangrui Meng <meng@databricks.com>
very minor update Ishiihara Author: Xiangrui Meng <meng@databricks.com> Closes apache#2043 from mengxr/minor-w2v and squashes the following commits: be649fd [Xiangrui Meng] remove map because we only need append eccefcc [Xiangrui Meng] minor updates to word2vec
Sorry I didn't realize this in apache#2043. Ishiihara Author: Xiangrui Meng <meng@databricks.com> Closes apache#2049 from mengxr/more-w2v and squashes the following commits: 050b1c5 [Xiangrui Meng] output shuffle data directly
very minor update @Ishiihara