Github user robert-dodier commented on the pull request:
https://github.com/apache/spark/pull/10611#issuecomment-169475870
OK, thanks for the heads-up. I have pushed commit 237677e which addresses
the style problems.
---
If your project is set up for it, you can reply to this email
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10611#issuecomment-169512288
**[Test build #2340 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2340/consoleFull)**
for PR 10611 at commit
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/10611#issuecomment-169512176
LGTM pending tests
---
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/10611#issuecomment-169518907
**[Test build #2340 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2340/consoleFull)**
for PR 10611 at commit
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/10611#issuecomment-169541007
Merging with master
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 project
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/10611
---
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 thunterdb commented on the pull request:
https://github.com/apache/spark/pull/10611#issuecomment-169483568
cc @jkbradley
---
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 robert-dodier opened a pull request:
https://github.com/apache/spark/pull/10611
[SPARK-12663] [MLlib] More informative error message in
MLUtils.loadLibSVMFile
This PR contains 1 commit which resolves
[SPARK-12663](https://issues.apache.org/jira/browse/SPARK-12663).
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/10611#issuecomment-169185779
Can one of the admins verify this patch?
---
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/10611#issuecomment-169190431
**[Test build #2332 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2332/consoleFull)**
for PR 10611 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/10611#issuecomment-169190664
**[Test build #2332 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/2332/consoleFull)**
for PR 10611 at commit
Github user thunterdb commented on a diff in the pull request:
https://github.com/apache/spark/pull/10611#discussion_r48919311
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -86,7 +86,7 @@ object MLUtils {
val indicesLength =
Github user thunterdb commented on the pull request:
https://github.com/apache/spark/pull/10611#issuecomment-169190994
@robert-dodier thanks for your PR. Can you please fix the style issue?
---
If your project is set up for it, you can reply to this email and have your
reply appear
13 matches
Mail list logo
