Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r35617723
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaOneVsRestExample.java
---
@@ -0,0 +1,236 @@
+/*
+ * Licensed to the Apache Software
Github user harsha2010 commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r35606801
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaOneVsRestExample.java
---
@@ -0,0 +1,236 @@
+/*
+ * Licensed to the Apache Softw
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r35606520
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaOneVsRestExample.java
---
@@ -0,0 +1,236 @@
+/*
+ * Licensed to the Apache Softwa
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r35548445
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaOneVsRestExample.java
---
@@ -0,0 +1,236 @@
+/*
+ * Licensed to the Apache Software
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/6115
---
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 jkbradley commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102565866
LGTM It worked fine when I ran it again just now. Thanks for the PR!
Merging into master and branch-1.4
---
If your project is set up for it, you can reply to this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102284174
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/6115#issuecomment-102284136
[Test build #32774 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32774/consoleFull)
for PR 6115 at commit
[`87ad3c7`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102284176
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102263667
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102263658
[Test build #32770 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32770/consoleFull)
for PR 6115 at commit
[`f5d9891`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102263666
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102258463
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102258462
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/6115#issuecomment-102258455
[Test build #32765 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32765/consoleFull)
for PR 6115 at commit
[`dfd660c`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102255735
[Test build #32774 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32774/consoleFull)
for PR 6115 at commit
[`87ad3c7`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102255002
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/6115#issuecomment-102255065
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/6115#issuecomment-102245261
[Test build #32770 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32770/consoleFull)
for PR 6115 at commit
[`f5d9891`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102245109
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/6115#issuecomment-102245119
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/6115#issuecomment-102243616
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102243610
[Test build #32766 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32766/consoleFull)
for PR 6115 at commit
[`7076084`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102243613
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102241899
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/6115#issuecomment-102241959
[Test build #32766 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32766/consoleFull)
for PR 6115 at commit
[`7076084`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102241889
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102241605
[Test build #32765 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32765/consoleFull)
for PR 6115 at commit
[`dfd660c`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102241533
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/6115#issuecomment-102241546
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 harsha2010 commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102241529
@jkbradley thanks for the feedback. i have refactored the examples based on
feedback. also thanks for pointing out the issue with mnist test data, i copied
your solut
Github user jkbradley commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102133363
@harsha2010 Those new comments are after trying out the examples on data.
One more small item on formatting results: It would be nice to have extra
newlines s
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30350138
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaOneVsRestExample.java
---
@@ -0,0 +1,225 @@
+/*
+ * Licensed to the Apache Softwa
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30350118
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaOneVsRestExample.java
---
@@ -0,0 +1,225 @@
+/*
+ * Licensed to the Apache Softwa
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30350140
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaOneVsRestExample.java
---
@@ -0,0 +1,225 @@
+/*
+ * Licensed to the Apache Softwa
Github user harsha2010 commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-102123933
Jenkins, retest 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
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101973576
[Test build #32687 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32687/consoleFull)
for PR 6115 at commit
[`8703e4f`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101973584
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101973583
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101951685
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/6115#issuecomment-101951642
[Test build #32682 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32682/consoleFull)
for PR 6115 at commit
[`cb23995`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101951689
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101949522
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101949524
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101949511
[Test build #32679 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32679/consoleFull)
for PR 6115 at commit
[`7f4e127`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101942224
[Test build #32687 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32687/consoleFull)
for PR 6115 at commit
[`8703e4f`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101941355
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/6115#issuecomment-101941409
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/6115#issuecomment-101934193
[Test build #32682 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32682/consoleFull)
for PR 6115 at commit
[`cb23995`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101933912
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/6115#issuecomment-101933903
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101916521
[Test build #32679 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32679/consoleFull)
for PR 6115 at commit
[`7f4e127`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101916383
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/6115#issuecomment-101916394
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/6115#issuecomment-101903109
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101903107
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/6115#issuecomment-101903100
[Test build #32670 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32670/consoleFull)
for PR 6115 at commit
[`d4c40d0`](https://git
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101882130
[Test build #32670 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32670/consoleFull)
for PR 6115 at commit
[`d4c40d0`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101882040
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/6115#issuecomment-101882054
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 jkbradley commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101841121
@harsha2010 Thanks for the PR! I like the examples and basically just had
small/formatting comments.
---
If your project is set up for it, you can reply to this emai
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30284064
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaOneVsRestExample.java
---
@@ -0,0 +1,182 @@
+/*
+ * Licensed to the Apache Softwa
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30284076
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/OneVsRestExample.scala ---
@@ -0,0 +1,169 @@
+/*
+ * Licensed to the Apache Software
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30284079
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/OneVsRestExample.scala ---
@@ -0,0 +1,169 @@
+/*
+ * Licensed to the Apache Software
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30284070
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaOneVsRestExample.java
---
@@ -0,0 +1,182 @@
+/*
+ * Licensed to the Apache Softwa
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30284067
--- Diff:
examples/src/main/java/org/apache/spark/examples/ml/JavaOneVsRestExample.java
---
@@ -0,0 +1,182 @@
+/*
+ * Licensed to the Apache Softwa
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30284078
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/OneVsRestExample.scala ---
@@ -0,0 +1,169 @@
+/*
+ * Licensed to the Apache Software
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30284087
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/OneVsRestExample.scala ---
@@ -0,0 +1,169 @@
+/*
+ * Licensed to the Apache Software
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30284085
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/OneVsRestExample.scala ---
@@ -0,0 +1,169 @@
+/*
+ * Licensed to the Apache Software
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30284083
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/OneVsRestExample.scala ---
@@ -0,0 +1,169 @@
+/*
+ * Licensed to the Apache Software
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30284081
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/OneVsRestExample.scala ---
@@ -0,0 +1,169 @@
+/*
+ * Licensed to the Apache Software
Github user jkbradley commented on a diff in the pull request:
https://github.com/apache/spark/pull/6115#discussion_r30284073
--- Diff:
examples/src/main/scala/org/apache/spark/examples/ml/OneVsRestExample.scala ---
@@ -0,0 +1,169 @@
+/*
+ * Licensed to the Apache Software
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101813584
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101813542
[Test build #32636 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32636/consoleFull)
for PR 6115 at commit
[`461eb38`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101813578
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/6115#issuecomment-101798734
[Test build #32632 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32632/consoleFull)
for PR 6115 at commit
[`e0106d9`](https://git
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101798755
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101798753
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/6115#issuecomment-101779634
[Test build #32636 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32636/consoleFull)
for PR 6115 at commit
[`461eb38`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101779434
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/6115#issuecomment-101779474
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/6115#issuecomment-101765575
[Test build #32632 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32632/consoleFull)
for PR 6115 at commit
[`e0106d9`](https://gith
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/6115#issuecomment-101764654
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/6115#issuecomment-101764706
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
84 matches
Mail list logo