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-7085][MLlib] Fix miniBatchFraction parameter in train method called with 4 arguments #5658

Closed
wants to merge 1 commit into from

Conversation

kuromt
Copy link

@kuromt kuromt commented Apr 23, 2015

No description provided.

@srowen
Copy link
Member

srowen commented Apr 23, 2015

ok to test

@srowen
Copy link
Member

srowen commented Apr 23, 2015

Given the comment and the case below this, this looks like a correct change.

@srowen
Copy link
Member

srowen commented Apr 23, 2015

@kuromatsu-nobuyuki the title of this isn't in the right format; review https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark

@kuromt
Copy link
Author

kuromt commented Apr 23, 2015

Sorry, can I rename this title?

@srowen
Copy link
Member

srowen commented Apr 23, 2015

Yes, just edit it right here.

@kuromt kuromt changed the title Fix miniBatchFraction parameter in train method called with 4 arguments [SPARK-7085][MLlib] Fix miniBatchFraction parameter in train method called with 4 arguments Apr 23, 2015
@kuromt
Copy link
Author

kuromt commented Apr 23, 2015

Thanks, I have edited.

@SparkQA
Copy link

SparkQA commented Apr 23, 2015

Test build #30831 has finished for PR 5658 at commit 6ec2d21.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds no public classes.
  • This patch does not change any dependencies.

@jkbradley
Copy link
Member

LGTM. The only place that was used was the unit test runRidgeRegressionUsingStaticMethods.
I've made a note to include this in the migration guide for Spark 1.4
Thanks! I'll merge it into master

@asfgit asfgit closed this in 3e91cc2 Apr 23, 2015
jeanlyn pushed a commit to jeanlyn/spark that referenced this pull request May 14, 2015
…alled with 4 arguments

Author: wizz <wizz@wizz-dev01.kawasaki.flab.fujitsu.com>

Closes apache#5658 from kuromatsu-nobuyuki/SPARK-7085 and squashes the following commits:

6ec2d21 [wizz] Fix miniBatchFraction parameter in train method called with 4 arguments
nemccarthy pushed a commit to nemccarthy/spark that referenced this pull request Jun 19, 2015
…alled with 4 arguments

Author: wizz <wizz@wizz-dev01.kawasaki.flab.fujitsu.com>

Closes apache#5658 from kuromatsu-nobuyuki/SPARK-7085 and squashes the following commits:

6ec2d21 [wizz] Fix miniBatchFraction parameter in train method called with 4 arguments
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