Github user nblintao commented on a diff in the pull request:
https://github.com/apache/spark/pull/14204#discussion_r71830047
--- Diff: core/src/test/scala/org/apache/spark/util/JsonProtocolSuite.scala
---
@@ -73,13 +73,15 @@ class JsonProtocolSuite extends SparkFunSuite {
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14204
**[Test build #62719 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62719/consoleFull)**
for PR 14204 at commit
Github user techaddict commented on the issue:
https://github.com/apache/spark/pull/14315
@jaceklaskowski thanks for finding this out. Its weird it passed locally
too.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14298
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62706/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14298
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14298
**[Test build #62706 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62706/consoleFull)**
for PR 14298 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14302
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14302
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62709/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14302
**[Test build #62709 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62709/consoleFull)**
for PR 14302 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14313
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62708/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14313
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14313
**[Test build #62708 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62708/consoleFull)**
for PR 14313 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14315
I'm wondering why that PR passed all tests...
LGTM, pending jenkins, thanks for finding this out!
---
If your project is set up for it, you can reply to this email and have your
reply
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/14210
---
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 jaceklaskowski commented on the issue:
https://github.com/apache/spark/pull/14315
/cc @cloud-fan @techaddict
---
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
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/14210
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 have this feature
enabled and
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14315
**[Test build #62718 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62718/consoleFull)**
for PR 14315 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13373
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62704/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13373
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 does not have this feature
enabled
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13373
**[Test build #62704 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62704/consoleFull)**
for PR 13373 at commit
GitHub user jaceklaskowski opened a pull request:
https://github.com/apache/spark/pull/14315
[HOTFIX] Fix annotation argument needs to be a constant
## What changes were proposed in this pull request?
Fix for compilation error:
```
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14210#discussion_r71828372
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/CreateTableAsSelectSuite.scala
---
@@ -212,7 +213,23 @@ class CreateTableAsSelectSuite
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14313
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62705/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14313
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14314
**[Test build #62717 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62717/consoleFull)**
for PR 14314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14313
**[Test build #62705 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62705/consoleFull)**
for PR 14313 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14285
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62703/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14285
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14285
**[Test build #62703 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62703/consoleFull)**
for PR 14285 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14314
**[Test build #62716 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62716/consoleFull)**
for PR 14314 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13373
**[Test build #62715 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62715/consoleFull)**
for PR 13373 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14132
**[Test build #62714 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62714/consoleFull)**
for PR 14132 at commit
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14314#discussion_r71827097
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -105,6 +105,12 @@ case class CreateViewCommand(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13373
**[Test build #62713 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62713/consoleFull)**
for PR 13373 at commit
Github user lianhuiwang commented on the issue:
https://github.com/apache/spark/pull/13373
cc @cloud-fan @rxin @hvanhovell
---
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 lianhuiwang commented on the issue:
https://github.com/apache/spark/pull/14285
cc @cloud-fan @rxin @hvanhovell
---
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 a diff in the pull request:
https://github.com/apache/spark/pull/14210#discussion_r71826808
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -344,6 +344,11 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14132#discussion_r71826514
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/catalyst/SQLBuilder.scala ---
@@ -193,6 +196,9 @@ class SQLBuilder(logicalPlan: LogicalPlan)
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/14314#discussion_r71825974
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -105,6 +105,12 @@ case class CreateViewCommand(
val
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14132#discussion_r71825927
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/catalyst/SQLBuilder.scala ---
@@ -193,6 +196,9 @@ class SQLBuilder(logicalPlan: LogicalPlan)
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14259#discussion_r71825821
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DatasetAggregatorSuite.scala ---
@@ -314,4 +343,14 @@ class DatasetAggregatorSuite extends
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14132#discussion_r71825810
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -339,8 +339,11 @@ class AstBuilder extends
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14290
The analyzer resolution requires them as children. Although this is
inconsistent with the other similar nodes, we still can tolerate it.
---
If your project is set up for it, you can reply to
Github user gatorsmile closed the pull request at:
https://github.com/apache/spark/pull/14290
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14314
**[Test build #62712 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62712/consoleFull)**
for PR 14314 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/14259#discussion_r71825205
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/DatasetAggregatorSuite.scala ---
@@ -314,4 +343,14 @@ class DatasetAggregatorSuite extends QueryTest
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/14314
[SPARK-16678] [SPARK-16677] Fix two View-related bugs
### What changes were proposed in this pull request?
**Issue 1: Disallow Creating a View when the same-name Table Exists**
When we
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14292
**[Test build #3189 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3189/consoleFull)**
for PR 14292 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13373
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13373
**[Test build #62711 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62711/consoleFull)**
for PR 13373 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13373
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62711/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13373
**[Test build #62711 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62711/consoleFull)**
for PR 13373 at commit
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14116#discussion_r71823904
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -484,8 +521,11 @@ class SessionCatalog(
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14116#discussion_r71823733
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/SessionCatalog.scala
---
@@ -471,6 +480,34 @@ class SessionCatalog(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13373
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13373
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62710/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13373
**[Test build #62710 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62710/consoleFull)**
for PR 13373 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14297
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62701/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14297
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 does not have this feature
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14132#discussion_r71823645
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/catalyst/SQLBuilder.scala ---
@@ -193,6 +196,9 @@ class SQLBuilder(logicalPlan: LogicalPlan)
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14132#discussion_r71823613
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -339,8 +339,11 @@ class AstBuilder extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14302
**[Test build #62709 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62709/consoleFull)**
for PR 14302 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13373
**[Test build #62710 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62710/consoleFull)**
for PR 13373 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14297
**[Test build #62701 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62701/consoleFull)**
for PR 14297 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13373
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62707/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13373
**[Test build #62707 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62707/consoleFull)**
for PR 13373 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13373
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 does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14313
**[Test build #62708 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62708/consoleFull)**
for PR 14313 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/14307
Hi, @hvanhovell .
Could you review this PR about `SQLBuilder`?
---
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 issue:
https://github.com/apache/spark/pull/14204
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 does not have this feature
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14132#discussion_r71823152
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/catalyst/SQLBuilder.scala ---
@@ -193,6 +196,9 @@ class SQLBuilder(logicalPlan: LogicalPlan)
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13373
**[Test build #62707 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62707/consoleFull)**
for PR 13373 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14204
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62700/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14204
**[Test build #62700 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62700/consoleFull)**
for PR 14204 at commit
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14132#discussion_r71823079
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/catalyst/SQLBuilder.scala ---
@@ -193,6 +196,9 @@ class SQLBuilder(logicalPlan: LogicalPlan)
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14132#discussion_r71822992
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -339,8 +339,11 @@ class AstBuilder extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14298
**[Test build #62706 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62706/consoleFull)**
for PR 14298 at commit
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/14132#discussion_r71822903
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -347,6 +347,16 @@ querySpecification
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14313
**[Test build #62705 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62705/consoleFull)**
for PR 14313 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14313
Could you please take a look here @cloud-fan and @yhuai ? This is happening
for writing too. I will open new one for writing as well.
---
If your project is set up for it, you can reply to
GitHub user HyukjinKwon opened a pull request:
https://github.com/apache/spark/pull/14313
[SPARK-16674][SQL] Avoid per-record type dispatch in JDBC when reading
## What changes were proposed in this pull request?
Currently, `JDBCRDD.compute` is doing type dispatch for each
Github user lw-lin commented on the issue:
https://github.com/apache/spark/pull/14298
@cloud-fan could you also help review this? 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 not have this
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14210#discussion_r71822711
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkSqlParser.scala ---
@@ -344,6 +344,11 @@ class SparkSqlAstBuilder(conf: SQLConf)
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14297#discussion_r71822672
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -88,23 +96,23 @@ case class CreateViewCommand(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14296
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14296
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62702/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14296
**[Test build #62702 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62702/consoleFull)**
for PR 14296 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/13373
**[Test build #62704 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62704/consoleFull)**
for PR 13373 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14297#discussion_r71821115
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -88,23 +96,23 @@ case class CreateViewCommand(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14285
**[Test build #62703 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62703/consoleFull)**
for PR 14285 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14297#discussion_r71820977
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -88,23 +96,23 @@ case class CreateViewCommand(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14132#discussion_r71820467
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/catalyst/SQLBuilder.scala ---
@@ -193,6 +196,9 @@ class SQLBuilder(logicalPlan: LogicalPlan)
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14132#discussion_r71820260
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -339,8 +339,11 @@ class AstBuilder extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14302
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 does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14302
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/62699/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14302
**[Test build #62699 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/62699/consoleFull)**
for PR 14302 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/14132#discussion_r71819403
--- Diff:
sql/catalyst/src/main/antlr4/org/apache/spark/sql/catalyst/parser/SqlBase.g4 ---
@@ -347,6 +347,16 @@ querySpecification
windows?)
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14297#discussion_r71819083
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -88,23 +96,23 @@ case class CreateViewCommand(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14297#discussion_r71818729
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/command/views.scala ---
@@ -88,23 +96,23 @@ case class CreateViewCommand(
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/14265
cc @jkbradley 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 not have this feature
enabled and
1 - 100 of 485 matches
Mail list logo
