Skip to content

[SPARK-10706] [MLLIB] Add java wrapper for random vector rdd#8841

Closed
rotationsymmetry wants to merge 1 commit intoapache:masterfrom
rotationsymmetry:SPARK-10706
Closed

[SPARK-10706] [MLLIB] Add java wrapper for random vector rdd#8841
rotationsymmetry wants to merge 1 commit intoapache:masterfrom
rotationsymmetry:SPARK-10706

Conversation

@rotationsymmetry
Copy link
Contributor

Add java wrapper for random vector rdd

@holdenk @srowen

@rotationsymmetry rotationsymmetry changed the title [SPARK-10626] [MLLIB] Add java wrapper for random vector rdd [SPARK-10706] [MLLIB] Add java wrapper for random vector rdd Sep 20, 2015
@SparkQA
Copy link

SparkQA commented Sep 20, 2015

Test build #42721 has finished for PR 8841 at commit 59a856f.

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

@holdenk
Copy link
Contributor

holdenk commented Sep 20, 2015

This LGTM, @srowen can hopefully take a look too.

@srowen
Copy link
Member

srowen commented Sep 20, 2015

LGTM

@srowen
Copy link
Member

srowen commented Sep 21, 2015

Sorry @rotationsymmetry, some conflicts now -- can you rebase and force push?

@SparkQA
Copy link

SparkQA commented Sep 21, 2015

Test build #42780 has finished for PR 8841 at commit 9258225.

  • This patch fails MiMa tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • class CountVectorizer(JavaEstimator, HasInputCol, HasOutputCol):
    • class CountVectorizerModel(JavaModel):
    • case class Sort(

@SparkQA
Copy link

SparkQA commented Sep 22, 2015

Test build #42795 has finished for PR 8841 at commit f669b6e.

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

@rotationsymmetry
Copy link
Contributor Author

@srowen I have rebased and pushed the PR. Thanks!

cc @holdenk

@asfgit asfgit closed this in 870b8a2 Sep 22, 2015
@rotationsymmetry rotationsymmetry deleted the SPARK-10706 branch September 23, 2015 04:39
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