Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/6538
---
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108262484
[Test build #34075 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34075/consoleFull)
for PR 6538 at commit
[`79d9c11`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108262500
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108225720
[Test build #34075 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34075/consoleFull)
for PR 6538 at commit
[`79d9c11`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108225274
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108225314
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 have
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31597847
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,17 @@ object MLUtils {
val value = indexAndValue(1).toD
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31596514
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,17 @@ object MLUtils {
val value = indexAndValue(1).toD
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31596512
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,17 @@ object MLUtils {
val value = indexAndValue(1).toD
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108178964
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108178957
[Test build #34046 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34046/consoleFull)
for PR 6538 at commit
[`4310710`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108156628
[Test build #34046 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34046/consoleFull)
for PR 6538 at commit
[`4310710`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108155636
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108155646
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 have
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108155579
[Test build #34045 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34045/consoleFull)
for PR 6538 at commit
[`96460f1`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108155580
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 project
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108154989
[Test build #34045 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34045/consoleFull)
for PR 6538 at commit
[`96460f1`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108153837
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 h
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108153877
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 have
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108097271
@hhbyyh this LGTM but can you rebase to resolve the merge conflict?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108065096
[Test build #34000 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34000/consoleFull)
for PR 6538 at commit
[`20a2811`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108065123
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 project does n
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108013554
[Test build #34000 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/34000/consoleFull)
for PR 6538 at commit
[`20a2811`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108013087
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 have this f
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108013062
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 have thi
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-108012118
test this please
---
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 featur
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107938527
**[Test build #33979 timed
out](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33979/consoleFull)**
for PR 6538 at commit
[`20a2811`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107938544
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 project does n
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107905302
[Test build #33979 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33979/consoleFull)
for PR 6538 at commit
[`20a2811`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107904738
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 have this f
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107904711
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 have thi
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31507411
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,19 @@ object MLUtils {
val value = indexAndValue(1).toD
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31505632
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,19 @@ object MLUtils {
val value = indexAndValue(1).toD
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31504366
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,19 @@ object MLUtils {
val value = indexAndValue(1).toD
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31490778
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,19 @@ object MLUtils {
val value = indexAndValue(1).toD
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31489051
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,19 @@ object MLUtils {
val value = indexAndValue(1).toD
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31472424
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,19 @@ object MLUtils {
val value = indexAndValue(1).toD
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107491755
[Test build #33887 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33887/consoleFull)
for PR 6538 at commit
[`6e4f8ca`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107491809
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 project does n
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107407349
[Test build #33887 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33887/consoleFull)
for PR 6538 at commit
[`6e4f8ca`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107406712
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 have this f
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107406673
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 have thi
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31394789
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,8 @@ object MLUtils {
val value = indexAndValue(1).toDo
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107225358
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 project does n
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107225342
[Test build #33855 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33855/consoleFull)
for PR 6538 at commit
[`9956365`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107200437
[Test build #33855 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33855/consoleFull)
for PR 6538 at commit
[`9956365`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107200324
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 have this f
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107200320
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 have thi
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31390798
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,8 @@ object MLUtils {
val value = indexAndValue(1).toDo
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31390517
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,8 @@ object MLUtils {
val value = indexAndValue(1).toDo
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31390381
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,8 @@ object MLUtils {
val value = indexAndValue(1).toDo
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31390293
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,8 @@ object MLUtils {
val value = indexAndValue(1).toDo
Github user hhbyyh commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107181910
@srowen. Great idea. I will add an unit test.
---
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 proj
Github user hhbyyh commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31390249
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,8 @@ object MLUtils {
val value = indexAndValue(1).toDo
Github user srowen commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107176670
In fact I'd make a test?
---
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 thi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107175835
[Test build #33851 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33851/consoleFull)
for PR 6538 at commit
[`5bd1f9a`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107175864
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 project
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6538#discussion_r31389800
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/util/MLUtils.scala ---
@@ -82,6 +82,8 @@ object MLUtils {
val value = indexAndValue(1).toDo
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107159291
[Test build #33851 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/33851/consoleFull)
for PR 6538 at commit
[`5bd1f9a`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107158807
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 have
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6538#issuecomment-107158794
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 h
GitHub user hhbyyh opened a pull request:
https://github.com/apache/spark/pull/6538
[Spark-7983] [MLlib] add require for one-based in loadLIBSVM
jira: https://issues.apache.org/jira/browse/SPARK-7983
Customers frequently use zero-based indices in their LIBSVM files. No
war
62 matches
Mail list logo