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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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 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
34 matches
Mail list logo