Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/5038
---
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 enab
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/5038#issuecomment-83105312
LGTM. Will wait a short time for more 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 pro
Github user yanboliang commented on the pull request:
https://github.com/apache/spark/pull/5038#issuecomment-82787376
@srowen , sorry for the misunderstand. I have change to use string
interpolation and delete ```toDebugString``` which may be useless.
---
If your project is set up fo
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5038#issuecomment-82780195
[Test build #28774 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28774/consoleFull)
for PR 5038 at commit
[`2f578b0`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5038#issuecomment-82780219
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5038#issuecomment-82746937
[Test build #28774 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28774/consoleFull)
for PR 5038 at commit
[`2f578b0`](https://githu
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/5038#issuecomment-82359681
Now this doesn't use string interpolation at all, which is the opposite of
what I suggested. I think it's better to use string interpolation, and use it
consistently. I al
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5038#issuecomment-82349773
[Test build #28719 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28719/consoleFull)
for PR 5038 at commit
[`78b33f2`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5038#issuecomment-82349810
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5038#issuecomment-82314396
[Test build #28719 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28719/consoleFull)
for PR 5038 at commit
[`78b33f2`](https://githu
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/5038#discussion_r26485785
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/regression/GeneralizedLinearAlgorithm.scala
---
@@ -76,7 +76,17 @@ abstract class GeneralizedLinearModel
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/5038#discussion_r26485698
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/regression/GeneralizedLinearAlgorithm.scala
---
@@ -76,7 +76,17 @@ abstract class GeneralizedLinearModel
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/5038#discussion_r26485545
--- Diff:
mllib/src/main/scala/org/apache/spark/mllib/classification/LogisticRegression.scala
---
@@ -163,6 +163,10 @@ class LogisticRegressionModel (
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/5038#issuecomment-81509233
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5038#issuecomment-81509196
[Test build #28640 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28640/consoleFull)
for PR 5038 at commit
[`1e8a023`](https://gith
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/5038#issuecomment-81464575
[Test build #28640 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/28640/consoleFull)
for PR 5038 at commit
[`1e8a023`](https://githu
GitHub user yanboliang opened a pull request:
https://github.com/apache/spark/pull/5038
[SPARK-6291] [MLLIB] GLM toString & toDebugString
GLM toString prints out intercept, numFeatures.
For LogisticRegression and SVM model, toString also prints out numClasses,
threshold.
GLM
17 matches
Mail list logo