Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12298
---
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 mengxr commented on the pull request:
https://github.com/apache/spark/pull/12298#issuecomment-208438237
LGTM. Merged into 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 does
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12298#issuecomment-208278992
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12298#issuecomment-208278993
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/12298#issuecomment-208278760
**[Test build #55518 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55518/consoleFull)**
for PR 12298 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12298#issuecomment-208234643
**[Test build #55518 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55518/consoleFull)**
for PR 12298 at commit
GitHub user dbtsai opened a pull request:
https://github.com/apache/spark/pull/12298
[SPARK-14462][ML][MLLIB] Add the mllib-local build to maven pom
## What changes were proposed in this pull request?
In order to separate the linear algebra, and vector matrix classes into a
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207854421
@dbtsai Let's remove the test dependency on spark-core:test and use
`FunSuite` with `// scalastyle:ignore funsuite` in mllib-local test, similar to
sketch.
---
If
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207854299
Reverted.
https://amplab.cs.berkeley.edu/jenkins/view/Spark%20QA%20Compile/job/spark-master-compile-maven-hadoop-2.7/950/consoleFull
~~~
[info] Compiling 1
Github user yhuai commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207817560
Looks like it breaks the build?
---
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 asfgit closed the pull request at:
https://github.com/apache/spark/pull/12241
---
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 mengxr commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207814814
LGTM. Merged into 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 does
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207707531
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/12241#issuecomment-207707525
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/12241#issuecomment-207707201
**[Test build #55414 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55414/consoleFull)**
for PR 12241 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207706848
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/12241#issuecomment-207706847
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/12241#issuecomment-207706526
**[Test build #55413 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55413/consoleFull)**
for PR 12241 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207690112
**[Test build #55414 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55414/consoleFull)**
for PR 12241 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207689839
**[Test build #55413 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55413/consoleFull)**
for PR 12241 at commit
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/12241#discussion_r59107331
--- Diff: mllib/pom.xml ---
@@ -63,25 +63,20 @@
${project.version}
- org.scalanlp
--- End diff --
Please
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207662093
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/12241#issuecomment-207662090
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/12241#issuecomment-207661880
**[Test build #55400 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55400/consoleFull)**
for PR 12241 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207632887
**[Test build #55400 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55400/consoleFull)**
for PR 12241 at commit
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207632164
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207632087
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/12241#issuecomment-207632079
**[Test build #55398 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55398/consoleFull)**
for PR 12241 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207632084
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 dbtsai commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207632064
This is the minimal change for creating a new jar build. Let's wait the
result of Jenkins. We'll move the code in a separate PR once this is merged.
Thanks.
---
If
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207631830
**[Test build #55398 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55398/consoleFull)**
for PR 12241 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207631086
Test FAILed.
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/12241#issuecomment-207631083
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/12241#issuecomment-207631071
**[Test build #55397 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55397/consoleFull)**
for PR 12241 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207630541
**[Test build #55397 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55397/consoleFull)**
for PR 12241 at commit
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/12241#discussion_r59079729
--- Diff: dev/sparktestsupport/modules.py ---
@@ -256,9 +256,21 @@ def __hash__(self):
)
+mllib_local = Module(
+
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/12241#discussion_r59081509
--- Diff: dev/sparktestsupport/modules.py ---
@@ -256,9 +256,21 @@ def __hash__(self):
)
+mllib_local = Module(
+
Github user dbtsai commented on a diff in the pull request:
https://github.com/apache/spark/pull/12241#discussion_r59081171
--- Diff: core/pom.xml ---
@@ -35,6 +35,11 @@
http://spark.apache.org/
+ org.apache.spark
--- End diff --
I
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/12241#discussion_r59080184
--- Diff: dev/sparktestsupport/modules.py ---
@@ -256,9 +256,21 @@ def __hash__(self):
)
+mllib_local = Module(
+
Github user mengxr commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207572941
@dbtsai Let us separate the project/build setup from the content. In this
PR, we only need one dummy source file under `mllib-local` with a dummy test to
ensure project
Github user mengxr commented on a diff in the pull request:
https://github.com/apache/spark/pull/12241#discussion_r59077858
--- Diff: dev/sparktestsupport/modules.py ---
@@ -256,9 +256,21 @@ def __hash__(self):
)
+mllib_local = Module(
+
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207395644
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207395645
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/12241#issuecomment-207395345
**[Test build #55340 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55340/consoleFull)**
for PR 12241 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/12241#discussion_r59008847
--- Diff: mllib-local/pom.xml ---
@@ -0,0 +1,105 @@
+
+
+
+http://maven.apache.org/POM/4.0.0;
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/12241#discussion_r59008785
--- Diff: core/pom.xml ---
@@ -35,6 +35,11 @@
http://spark.apache.org/
+ org.apache.spark
--- End diff --
This
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207351119
**[Test build #55340 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55340/consoleFull)**
for PR 12241 at commit
Github user dbtsai commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207349784
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207344297
Test FAILed.
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/12241#issuecomment-207344294
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/12241#issuecomment-207344017
**[Test build #55334 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55334/consoleFull)**
for PR 12241 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207273008
**[Test build #55334 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55334/consoleFull)**
for PR 12241 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207211022
Test FAILed.
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/12241#issuecomment-207211020
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/12241#issuecomment-207203766
**[Test build #55282 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55282/consoleFull)**
for PR 12241 at commit
Github user holdenk commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207148128
@dbtsai Did you generate docs for this PR? I'm not sure how docs handles
duplicate package classes between the components (might be ok but just
something worth making
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207146736
**[Test build #55282 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55282/consoleFull)**
for PR 12241 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207145929
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207145931
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/12241#issuecomment-207145916
**[Test build #55277 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55277/consoleFull)**
for PR 12241 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207141209
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207141212
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/12241#issuecomment-207141430
**[Test build #55277 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55277/consoleFull)**
for PR 12241 at commit
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207140896
**[Test build #55249 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55249/consoleFull)**
for PR 12241 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207133442
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/12241#issuecomment-207133431
**[Test build #55263 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55263/consoleFull)**
for PR 12241 at commit
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12241#issuecomment-207133438
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/12241#issuecomment-207132345
**[Test build #55263 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/55263/consoleFull)**
for PR 12241 at commit
68 matches
Mail list logo
