[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-24 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8369#discussion_r37773632 --- Diff: mllib/src/main/scala/org/apache/spark/mllib/tree/configuration/Strategy.scala --- @@ -158,6 +158,11 @@ class Strategy ( s Valid values

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-24 Thread mengxr
Github user mengxr commented on a diff in the pull request: https://github.com/apache/spark/pull/8369#discussion_r37773624 --- Diff: docs/ml-decision-tree.md --- @@ -416,61 +413,49 @@ DataFrame testData = splits[1]; // Train a DecisionTree model.

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-24 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-134298575 LGTM except some minor comments. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-134394833 [Test build #41476 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41476/console) for PR 8369 at commit

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-134389778 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-134395015 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-134395010 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-24 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-134391597 [Test build #41476 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41476/consoleFull) for PR 8369 at commit

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-24 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-134389628 OK I think that does it. Thanks for reviewing! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-24 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-134389802 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-24 Thread mengxr
Github user mengxr commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-134401679 Merged into master and branch-1.5. Thanks! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-24 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/8369 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature is

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133985112 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133985160 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133985688 [Test build #41433 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41433/consoleFull) for PR 8369 at commit

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133995245 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-23 Thread yanboliang
Github user yanboliang commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-134048598 Looks good! --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-23 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133995065 [Test build #41433 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41433/console) for PR 8369 at commit

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-23 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133995244 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread jkbradley
GitHub user jkbradley opened a pull request: https://github.com/apache/spark/pull/8369 [SPARK-10061] [DOC] ML ensemble docs User guide for spark.ml GBTs and Random Forests. The examples are copied from the decision tree guide and modified to run. I caught some issues I

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133580986 Note: This is blocked by [https://github.com/apache/spark/pull/8367] --- If your project is set up for it, you can reply to this email and have your reply appear on

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133581440 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133581455 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133581819 [Test build #41393 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41393/consoleFull) for PR 8369 at commit

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133583692 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133583606 [Test build #41393 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41393/console) for PR 8369 at commit

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133583694 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133591565 [Test build #41395 has started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41395/consoleFull) for PR 8369 at commit

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133589837 Merged build started. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133589779 Merged build triggered. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread SparkQA
Github user SparkQA commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133595820 [Test build #41395 has finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/41395/console) for PR 8369 at commit

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133596049 Test PASSed. Refer to this link for build results (access rights to CI server needed):

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread AmplabJenkins
Github user AmplabJenkins commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133596046 Merged build finished. Test PASSed. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] spark pull request: [SPARK-10061] [DOC] ML ensemble docs

2015-08-21 Thread jkbradley
Github user jkbradley commented on the pull request: https://github.com/apache/spark/pull/8369#issuecomment-133589437 Rebased to include GBT fix in master. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project