Github user keypointt commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-214832141
ping @mengxr
---
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 keypointt commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-213578574
@yinxusen No problem, I totally understand you guys are super busy ð
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-213575408
@mengxr LGTM
---
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 yinxusen commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-213571733
@keypointt Sorry for the late. I'm taking a look now.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well.
Github user keypointt commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-213543877
@yinxusen I've added back the files you mention above, and also updated the
description (top comment) in this PR. In this PR I only deleted couple files,
is there
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-210221791
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/12195#issuecomment-210221785
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/12195#issuecomment-210221606
**[Test build #55876 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55876/consoleFull)**
for PR 12195 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-210217502
**[Test build #55876 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55876/consoleFull)**
for PR 12195 at commit
Github user keypointt commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-210216484
@yinxusen I've added back the files you mention above, and also updated the
description (top comment) in this PR
---
If your project is set up for it, you can reply
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-210212578
Test FAILed.
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/12195#issuecomment-210212573
**[Test build #55873 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55873/consoleFull)**
for PR 12195 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-210212575
Merged build finished. Test FAILed.
---
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/12195#issuecomment-210212313
**[Test build #55873 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55873/consoleFull)**
for PR 12195 at commit
Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-210205656
And for examples in **Unsure code duplications (need doube check)** you
mentioned above, I think we should keep them, too.
---
If your project is set up for it, you
Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-210205433
So I think we should keep
* mllib/DecisionTreeRunner.scala
* mllib/GradientBoostedTreesRunner.scala
* mllib/DenseKMeans.scala
*
Github user yinxusen commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-210204569
@keypointt As @mengxr said here
https://github.com/apache/spark/pull/12092#issuecomment-204276885, we should
keep examples such as `DecisionTreeRunner`.
---
If your
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-206107020
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/12195#issuecomment-206106967
**[Test build #55083 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55083/consoleFull)**
for PR 12195 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-206107019
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 keypointt commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-206102900
Unsure code duplications (need doube check)
- AbstractParams.scala - cannot delete this one, not duplicate, its usage
found in many examples
-
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12195#issuecomment-206101554
**[Test build #55083 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55083/consoleFull)**
for PR 12195 at commit
GitHub user keypointt opened a pull request:
https://github.com/apache/spark/pull/12195
[Spark-14300][Docs][MLLIB]Scala MLlib examples code merge and clean up
## What changes were proposed in this pull request?
https://issues.apache.org/jira/browse/SPARK-14300
23 matches
Mail list logo