Github user davies commented on the issue:
https://github.com/apache/spark/pull/11956
@kiszk The current implementation use ByteBuffer and smart compression
algorithms, it too slow to build the in-memory cache, make it useless. So we'd
like to improve the performance of building
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/9571
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 feature
enabled and wishes so, or
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/14531
That sounds good to me! Let me update the code now.
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/13065
**[Test build #64579 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64579/consoleFull)**
for PR 13065 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/9571
@steveloughran you still have not addressed my comment about
`appUiLoadTimer`. The metric is still being registered even though it doesn't
make any sense to the user.
---
If your project is set up
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14638
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64569/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14638
**[Test build #64569 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64569/consoleFull)**
for PR 14638 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
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 hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14866#discussion_r76665431
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1534,6 +1542,34 @@ object
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14803
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64565/
Test PASSed.
---
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/14861
ok to test
---
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
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/1#discussion_r76667084
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -172,60 +172,47 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14855
**[Test build #64560 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64560/consoleFull)**
for PR 14855 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14118
**[Test build #64576 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64576/consoleFull)**
for PR 14118 at commit
Github user ajbozarth commented on a diff in the pull request:
https://github.com/apache/spark/pull/14835#discussion_r76668460
--- Diff:
core/src/test/scala/org/apache/spark/deploy/history/HistoryServerSuite.scala ---
@@ -100,6 +100,7 @@ class HistoryServerSuite extends
Github user felixcheung commented on a diff in the pull request:
https://github.com/apache/spark/pull/14859#discussion_r76668581
--- Diff: dev/appveyor-install-dependencies.ps1 ---
@@ -0,0 +1,145 @@
+<#
+Licensed to the Apache Software Foundation (ASF) under one
+or
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14118
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64576/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14118
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/13704
**[Test build #64594 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64594/consoleFull)**
for PR 13704 at commit
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14775
Thanks @felixcheung - Addressed both the comments. Let me know if this
looks good.
---
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 shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14775#discussion_r76654300
--- Diff: R/pkg/R/backend.R ---
@@ -25,9 +25,23 @@ isInstanceOf <- function(jobj, className) {
callJMethod(cls, "isInstance", jobj)
}
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/9571
**[Test build #64586 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64586/consoleFull)**
for PR 9571 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14865
**[Test build #64587 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64587/consoleFull)**
for PR 14865 at commit
Github user davies commented on the issue:
https://github.com/apache/spark/pull/14607
Merging into master and 2.0 branch.
---
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 hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/1#discussion_r76657914
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -172,60 +172,47 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14775
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/14775
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64585/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14860
**[Test build #64555 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64555/consoleFull)**
for PR 14860 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14691
**[Test build #64568 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64568/consoleFull)**
for PR 14691 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14861
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/14527
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64571/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14861
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64591/
Test FAILed.
---
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/14835#discussion_r76667335
--- Diff:
core/src/test/scala/org/apache/spark/deploy/history/HistoryServerSuite.scala ---
@@ -100,6 +100,7 @@ class HistoryServerSuite extends
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/14864
Jenkins test this please. The last run had JVM crash
---
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 ajbozarth commented on a diff in the pull request:
https://github.com/apache/spark/pull/14835#discussion_r76667324
--- Diff: docs/monitoring.md ---
@@ -119,6 +119,14 @@ The history server can be configured as follows:
+
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/14835
yep, I believe you are correct, and if so I think we should just update the
docs for it to describe that. If you don't mind doing it along with this that
would be great.
---
If your project is
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/14835#discussion_r76669710
--- Diff:
core/src/test/scala/org/apache/spark/deploy/history/HistoryServerSuite.scala ---
@@ -100,6 +100,7 @@ class HistoryServerSuite extends
Github user eyalfa commented on the issue:
https://github.com/apache/spark/pull/1
@hvanhovell,please have a look
---
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 davies commented on the issue:
https://github.com/apache/spark/pull/11956
cc @rxin
---
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
Github user shivaram commented on the issue:
https://github.com/apache/spark/pull/14859
Good point on only running this on the master branch. We could even run it
periodically (say nightly) instead of on every commit.
---
If your project is set up for it, you can reply to this email
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14856
**[Test build #64559 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64559/consoleFull)**
for PR 14856 at commit
GitHub user maropu opened a pull request:
https://github.com/apache/spark/pull/14865
[SPARK-17289][SQL] Fix a bug to satisfy sort requirements in partial
aggregations
## What changes were proposed in this pull request?
Partial aggregations are generated in `EnsureRequirements`,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13065
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/13065
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64579/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14638
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 hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/1#discussion_r76662951
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -172,60 +172,47 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14531
**[Test build #64589 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64589/consoleFull)**
for PR 14531 at commit
Github user saurabhdaftary commented on the issue:
https://github.com/apache/spark/pull/2633
This would be a very useful feature to add-on. As @boy-uber stated would
love to see this getting in?
---
If your project is set up for it, you can reply to this email and have your
reply
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14866
**[Test build #64590 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64590/consoleFull)**
for PR 14866 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14803
**[Test build #64565 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64565/consoleFull)**
for PR 14803 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14860
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/14860
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64555/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14527
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/14861
**[Test build #64591 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64591/consoleFull)**
for PR 14861 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14623
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64570/
Test PASSed.
---
Github user eyalfa commented on a diff in the pull request:
https://github.com/apache/spark/pull/1#discussion_r7950
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -172,60 +172,47 @@ case class
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14623
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 tgravescs commented on the issue:
https://github.com/apache/spark/pull/14718
ping @vanzin
---
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
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14854
**[Test build #64593 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64593/consoleFull)**
for PR 14854 at commit
Github user JoshRosen commented on the issue:
https://github.com/apache/spark/pull/14859
It looks like a few other Apache projects use AppVeyor and it looks like
there's a designated process for asking INFRA to enable it for a GitHub mirror:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14858
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/14858
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64557/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14856
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 shivaram commented on a diff in the pull request:
https://github.com/apache/spark/pull/14775#discussion_r76654243
--- Diff: R/pkg/R/backend.R ---
@@ -37,12 +51,42 @@ callJMethod <- function(objId, methodName, ...) {
invokeJava(isStatic = FALSE, objId$id,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14856
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64559/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14775
**[Test build #64585 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64585/consoleFull)**
for PR 14775 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/1#discussion_r76656778
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -172,60 +172,47 @@ case class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/1#discussion_r76657472
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -172,60 +172,47 @@ case class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14775
**[Test build #64585 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64585/consoleFull)**
for PR 14775 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/1#discussion_r76663119
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CodeGenerationSuite.scala
---
@@ -124,7 +124,12 @@ class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/1#discussion_r76663187
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/ComplexTypeSuite.scala
---
@@ -174,7 +174,10 @@ class ComplexTypeSuite
Github user srinathshankar commented on the issue:
https://github.com/apache/spark/pull/14866
@rxin @sameeragarwal @hvanhovell @davies @JoshRosen
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/14862
**[Test build #64554 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64554/consoleFull)**
for PR 14862 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/14866#discussion_r76665008
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
---
@@ -1534,6 +1542,34 @@ object
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14862
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64554/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14861
**[Test build #64591 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64591/consoleFull)**
for PR 14861 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14527
**[Test build #64571 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64571/consoleFull)**
for PR 14527 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14862
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/14691
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/14691
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64568/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14864
**[Test build #64592 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64592/consoleFull)**
for PR 14864 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14426
**[Test build #64574 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64574/consoleFull)**
for PR 14426 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14858
**[Test build #64557 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64557/consoleFull)**
for PR 14858 at commit
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/1#discussion_r76657836
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -172,60 +172,47 @@ case class
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8880#discussion_r76658252
--- Diff:
yarn/src/test/scala/org/apache/spark/deploy/yarn/IOEncryptionSuite.scala ---
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apache Software
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14746
**[Test build #64588 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64588/consoleFull)**
for PR 14746 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/14452
**[Test build #64572 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/64572/consoleFull)**
for PR 14452 at commit
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/8880#discussion_r76658383
--- Diff:
yarn/src/test/scala/org/apache/spark/deploy/yarn/IOEncryptionSuite.scala ---
@@ -0,0 +1,110 @@
+/*
+ * Licensed to the Apache Software
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14452
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64572/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/14452
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 vanzin commented on the issue:
https://github.com/apache/spark/pull/8880
LGTM. Any remaining comments @zsxwing ?
---
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/14435
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/14435
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/64573/
Test FAILed.
---
Github user dhruve commented on the issue:
https://github.com/apache/spark/pull/14710
@angolon - Kindly resolve the conflicts and update the PR.
---
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/14607
---
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 eyalfa commented on a diff in the pull request:
https://github.com/apache/spark/pull/1#discussion_r76662599
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -172,60 +172,47 @@ case class
GitHub user srinathshankar opened a pull request:
https://github.com/apache/spark/pull/14866
[SPARK-17298][SQL] Require explicit CROSS join for cartesian products
## What changes were proposed in this pull request?
Require the use of CROSS join syntax in SQL (and a new
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/1#discussion_r76663028
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/analysis/AnalysisSuite.scala
---
@@ -218,9 +221,44 @@ class AnalysisSuite extends
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/14801#discussion_r76663011
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/HiveExternalCatalog.scala ---
@@ -450,7 +455,11 @@ private[spark] class
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/1#discussion_r76663726
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/complexTypeCreator.scala
---
@@ -172,60 +172,47 @@ case class
1 - 100 of 616 matches
Mail list logo
