Github user davies commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-201124959
I will fix this in #11828
---
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 davies commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-201124385
@gatorsmile This PR can't be easily reverted, so could you send a PR to
fix it?
---
If your project is set up for it, you can reply to this email and have your
reply a
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-201122555
@gatorsmile @cloud-fan This PR revert the change in
https://github.com/apache/spark/pull/3674, unfortunately the unit test in
AnalysisSuite. This test break once we enf
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199555171
thanks! merging to master!
---
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 ha
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/11208
---
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199420301
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199420306
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199419884
**[Test build #53685 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53685/consoleFull)**
for PR 11208 at commit
[`50abeec`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199374933
**[Test build #53685 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53685/consoleFull)**
for PR 11208 at commit
[`50abeec`](https://gi
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56854915
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -1934,6 +1934,50 @@ class SQLQuerySuite extends QueryTest with
SharedSQ
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199179852
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199179856
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199179614
**[Test build #53661 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53661/consoleFull)**
for PR 11208 at commit
[`0fce075`](https://g
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56789051
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -1934,6 +1934,50 @@ class SQLQuerySuite extends QueryTest with
SharedSQL
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199150315
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199150318
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199149855
**[Test build #53655 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53655/consoleFull)**
for PR 11208 at commit
[`ba3fe7c`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199149411
**[Test build #53661 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53661/consoleFull)**
for PR 11208 at commit
[`0fce075`](https://gi
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56785749
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -369,28 +370,83 @@ class Analyzer(
}
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56785740
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -737,6 +737,14 @@ class SQLQuerySuite extends QueryTest
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56785743
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -1934,6 +1934,43 @@ class SQLQuerySuite extends QueryTest with
SharedSQ
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56783882
--- Diff:
sql/hive/src/test/scala/org/apache/spark/sql/hive/execution/SQLQuerySuite.scala
---
@@ -737,6 +737,14 @@ class SQLQuerySuite extends QueryTest w
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56783810
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -1934,6 +1934,43 @@ class SQLQuerySuite extends QueryTest with
SharedSQL
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56783741
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -369,28 +370,83 @@ class Analyzer(
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56783697
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -369,28 +370,83 @@ class Analyzer(
}
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199128149
**[Test build #53655 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53655/consoleFull)**
for PR 11208 at commit
[`ba3fe7c`](https://gi
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199126672
@cloud-fan Thank you for your detailed reviews! I know all of you are very
busy. Let me know if anything needs a change. Thanks again!
---
If your project is set up
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56782647
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -369,28 +370,83 @@ class Analyzer(
}
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56781821
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -369,28 +370,83 @@ class Analyzer(
}
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56781817
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -1934,6 +1934,21 @@ class SQLQuerySuite extends QueryTest with
SharedSQ
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56781820
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -1934,6 +1934,21 @@ class SQLQuerySuite extends QueryTest with
SharedSQ
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56781815
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -369,28 +370,83 @@ class Analyzer(
}
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-199090151
Sorry for putting it here for such a long time, overall LGTM, will merge it
after you address the new comments, thanks!
---
If your project is set up for it, you can
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56779097
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -1934,6 +1934,21 @@ class SQLQuerySuite extends QueryTest with
SharedSQL
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r56779046
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -369,28 +370,83 @@ class Analyzer(
}
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-197606557
**[Test build #53376 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53376/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://gi
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-197634039
**[Test build #53376 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53376/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-197634281
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 gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-198794464
cc @yhuai
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-198775207
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-198775210
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-198775144
**[Test build #53620 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53620/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-198759811
**[Test build #53620 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53620/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://gi
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-198759454
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 have this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-197634280
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 gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-197635547
cc @yhuai
---
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
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-197605174
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 have this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-195792867
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-195792869
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-195792810
**[Test build #53008 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53008/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-195777551
**[Test build #53008 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/53008/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://gi
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-195777465
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 have this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-195569069
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/11208#issuecomment-195569043
**[Test build #52952 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52952/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://g
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-195569065
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/11208#issuecomment-195567000
**[Test build #52952 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52952/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://gi
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-195565781
cc @yhuai @cloud-fan
---
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 t
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-195565806
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 have this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-194170203
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/11208#issuecomment-194170195
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/11208#issuecomment-194169593
**[Test build #52727 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52727/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-194138373
**[Test build #52727 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52727/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://gi
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-194136402
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 have this
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-192036455
cc @yhuai : )
---
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 feat
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-191268943
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/11208#issuecomment-191268934
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/11208#issuecomment-191268151
**[Test build #52318 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52318/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-191232077
**[Test build #52318 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52318/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://gi
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-191230890
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 have this
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-189583957
cc @yhuai
---
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
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-189578223
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-189578225
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-189578179
**[Test build #52097 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52097/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-189567010
**[Test build #52097 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/52097/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://gi
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-189566461
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 have thi
Github user yhuai commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r54048383
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -369,28 +370,83 @@ class Analyzer(
}
/*
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-188141711
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/11208#issuecomment-188141707
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/11208#issuecomment-188141242
**[Test build #51854 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51854/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-188113972
**[Test build #51854 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51854/consoleFull)**
for PR 11208 at commit
[`e060dea`](https://gi
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r53900103
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -350,28 +351,83 @@ class Analyzer(
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r53899877
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -350,28 +351,83 @@ class Analyzer(
}
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r53896443
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -350,28 +351,83 @@ class Analyzer(
}
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r53891726
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -350,28 +351,83 @@ class Analyzer(
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r53890012
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -350,28 +351,83 @@ class Analyzer(
}
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r53889923
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -350,28 +351,83 @@ class Analyzer(
}
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-187559355
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-187559358
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-187558491
**[Test build #51729 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51729/consoleFull)**
for PR 11208 at commit
[`6b2d609`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-187528976
**[Test build #51729 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51729/consoleFull)**
for PR 11208 at commit
[`6b2d609`](https://gi
Github user gatorsmile commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-187526529
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 have this
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-187520700
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/11208#issuecomment-187520703
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r53734588
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -1938,6 +1938,21 @@ class SQLQuerySuite extends QueryTest with
SharedSQ
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-187464975
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 AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-187464976
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 SparkQA commented on the pull request:
https://github.com/apache/spark/pull/11208#issuecomment-187464748
**[Test build #51701 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/51701/consoleFull)**
for PR 11208 at commit
[`2c72edf`](https://g
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r53722350
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala
---
@@ -1938,6 +1938,21 @@ class SQLQuerySuite extends QueryTest with
SharedSQL
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r53722134
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameComplexTypeSuite.scala ---
@@ -30,6 +30,7 @@ class DataFrameComplexTypeSuite extends QueryT
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/11208#discussion_r53722149
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DataFrameComplexTypeSuite.scala ---
@@ -42,6 +43,7 @@ class DataFrameComplexTypeSuite extends QueryT
1 - 100 of 118 matches
Mail list logo