Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12634#discussion_r60848581
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -412,20 +412,26 @@ class Analyzer(
catalog.look
Github user tedyu commented on a diff in the pull request:
https://github.com/apache/spark/pull/12634#discussion_r60845292
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -412,20 +412,26 @@ class Analyzer(
catalog.loo
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/12634
---
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 rxin commented on the pull request:
https://github.com/apache/spark/pull/12634#issuecomment-213818904
Merging in 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 have this feat
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12634#issuecomment-213817224
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/12634#issuecomment-213817223
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/12634#issuecomment-213817154
**[Test build #56805 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56805/consoleFull)**
for PR 12634 at commit
[`cddc66b`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12634#issuecomment-213797489
**[Test build #56805 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56805/consoleFull)**
for PR 12634 at commit
[`cddc66b`](https://gi
Github user rxin commented on a diff in the pull request:
https://github.com/apache/spark/pull/12634#discussion_r60832345
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystConf.scala ---
@@ -32,6 +32,8 @@ trait CatalystConf {
def optimizerInSetConve
Github user cloud-fan commented on the pull request:
https://github.com/apache/spark/pull/12634#issuecomment-213707877
LGTM
---
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
enable
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/12634#discussion_r60826409
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/CatalystConf.scala ---
@@ -32,6 +32,8 @@ trait CatalystConf {
def optimizerInSet
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12634#issuecomment-213694869
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/12634#issuecomment-213694868
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/12634#issuecomment-213694843
**[Test build #56789 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56789/consoleFull)**
for PR 12634 at commit
[`a1980dc`](https://g
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12634#issuecomment-213681559
**[Test build #56789 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56789/consoleFull)**
for PR 12634 at commit
[`a1980dc`](https://gi
Github user AmplabJenkins commented on the pull request:
https://github.com/apache/spark/pull/12634#issuecomment-213679032
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/12634#issuecomment-213679029
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/12634#issuecomment-213679008
**[Test build #56787 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56787/consoleFull)**
for PR 12634 at commit
[`43cc738`](https://g
Github user davies commented on the pull request:
https://github.com/apache/spark/pull/12634#issuecomment-213678447
LGTM
---
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 a
Github user SparkQA commented on the pull request:
https://github.com/apache/spark/pull/12634#issuecomment-213678085
**[Test build #56787 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/56787/consoleFull)**
for PR 12634 at commit
[`43cc738`](https://gi
Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/12634#issuecomment-213677747
cc @davies / @andrewor14
---
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 rxin opened a pull request:
https://github.com/apache/spark/pull/12634
[SPARK-14869][SQL] Don't mask exceptions in ResolveRelations
## What changes were proposed in this pull request?
In order to support SPARK-11197 (run SQL directly on files), we added some
code in
22 matches
Mail list logo
