Skip to content

[SPARK-5939][MLLib] make FPGrowth example app take parameters #4714

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.

Sign up for GitHub

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

Jump to bottom
Closed
jackylk wants to merge 3 commits into from
Closed

[SPARK-5939][MLLib] make FPGrowth example app take parameters #4714

jackylk wants to merge 3 commits into from

Conversation

@jackylk
Copy link
Contributor

@jackylk jackylk commented Feb 21, 2015

Add parameter parsing in FPGrowth example app in Scala and Java
And a sample data file is added in data/mllib folder

@SparkQA
Copy link

SparkQA commented Feb 21, 2015

Test build #27814 has started for PR 4714 at commit 3bb74f6.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 21, 2015

Test build #27814 has finished for PR 4714 at commit 3bb74f6.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class Params(

@AmplabJenkins
Copy link

AmplabJenkins commented Feb 21, 2015

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27814/
Test PASSed.

mengxr
mengxr reviewed Feb 21, 2015
View reviewed changes
examples/src/main/java/org/apache/spark/examples/mllib/JavaFPGrowthExample.java Outdated
Copy link
Contributor

@mengxr mengxr Feb 21, 2015

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

run-example doesn't need org.apache.spark.examples. So this could be a one-liner:

.bin/run-example mllib.JavaFPGrowthExample ./data/mllib/sample_fpgrowth.txt

@SparkQA
Copy link

SparkQA commented Feb 21, 2015

Test build #27821 has started for PR 4714 at commit 8c478b3.

  • This patch merges cleanly.

@SparkQA
Copy link

SparkQA commented Feb 21, 2015

Test build #27821 has finished for PR 4714 at commit 8c478b3.

  • This patch passes all tests.
  • This patch merges cleanly.
  • This patch adds the following public classes (experimental):
    • case class Params(

@AmplabJenkins
Copy link

AmplabJenkins commented Feb 21, 2015

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/27821/
Test PASSed.

asfgit pushed a commit that referenced this pull request Feb 23, 2015
@mengxr
[SPARK-5939][MLLib] make FPGrowth example app take parameters
33b9084 
Add parameter parsing in FPGrowth example app in Scala and Java
And a sample data file is added in data/mllib folder

Author: Jacky Li <[email protected]>

Closes #4714 from jackylk/parameter and squashes the following commits:

8c478b3 [Jacky Li] fix according to comments
3bb74f6 [Jacky Li] make FPGrowth exampl app take parameters
f0e4d10 [Jacky Li] make FPGrowth exampl app take parameters

(cherry picked from commit 651a1c0)
Signed-off-by: Xiangrui Meng <[email protected]>
@mengxr
Copy link
Contributor

mengxr commented Feb 23, 2015

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

@asfgit asfgit closed this in 651a1c0 Feb 23, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@jackylk @SparkQA @AmplabJenkins @mengxr