Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12907
---
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 ena
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-218243000
OK, this looks good. I'm going to merge this into master 2.0 first so you
don't have to keep rebasing it. If there are problems we can fix them in
follow-up patches
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/12907#discussion_r62722850
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -66,7 +66,8 @@ class DDLSuite extends QueryTest with Share
Github user techaddict commented on a diff in the pull request:
https://github.com/apache/spark/pull/12907#discussion_r62617908
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -66,7 +66,8 @@ class DDLSuite extends QueryTest with Share
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/12907#discussion_r62595907
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/command/DDLSuite.scala
---
@@ -66,7 +66,8 @@ class DDLSuite extends QueryTest with Share
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-218021053
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-218021051
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-218020889
**[Test build #58175 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58175/consoleFull)**
for PR 12907 at commit
[`7787227`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-218004783
**[Test build #58175 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58175/consoleFull)**
for PR 12907 at commit
[`7787227`](https://gi
Github user techaddict commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-218003214
@andrewor14 test failed because `WAREHOUSE_PATH` is set to
`${system:user.dir}/spark-warehouse` by default
(https://github.com/apache/spark/blob/master/sql/core/src/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217992800
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217992797
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217992610
**[Test build #58168 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58168/consoleFull)**
for PR 12907 at commit
[`6b61b21`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217974127
**[Test build #58168 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58168/consoleFull)**
for PR 12907 at commit
[`6b61b21`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217968906
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217968874
**[Test build #58164 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58164/consoleFull)**
for PR 12907 at commit
[`405540c`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217968904
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217965357
**[Test build #58164 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58164/consoleFull)**
for PR 12907 at commit
[`405540c`](https://gi
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217955033
(you also need to fix the imports in many other files)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217948360
**[Test build #58155 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58155/consoleFull)**
for PR 12907 at commit
[`3baecf6`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217948368
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 projec
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217948372
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217947849
**[Test build #58155 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58155/consoleFull)**
for PR 12907 at commit
[`3baecf6`](https://gi
Github user andrewor14 commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217940272
@techaddict the changes look OK. I noticed there were a few places where we
didn't actually use `SparkSession` but just renamed the variable. Can you fix
those?
--
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/12907#discussion_r62543264
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/feature/ChiSqSelectorSuite.scala ---
@@ -30,8 +30,8 @@ class ChiSqSelectorSuite extends SparkFunSuite w
Github user andrewor14 commented on a diff in the pull request:
https://github.com/apache/spark/pull/12907#discussion_r62543080
--- Diff:
mllib/src/test/java/org/apache/spark/ml/classification/JavaOneVsRestSuite.java
---
@@ -17,69 +17,72 @@
package org.apache.spark.ml.c
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217649181
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217649180
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 projec
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12907#issuecomment-217649127
**[Test build #58071 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/58071/consoleFull)**
for PR 12907 at commit
[`5f23f3d`](https://g
29 matches
Mail list logo