Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17931
at the time we received `SQLMetrics` in `SQLListener` with task end event,
the registered accumulator may already be GCed, then there is no way to
retrieve the accumulator names, except we sending
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17961
Can one of the admins verify this patch?
---
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 guoxiaolongzte opened a pull request:
https://github.com/apache/spark/pull/17961
[SPARK-20720][WEB-UI]'Executor Summary' should show the exact number,
'Removed Executors' should display the specific number, in the Application Page
## What changes were proposed in this pu
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/17959
@gatorsmile Right, thanks for pointing this out!
---
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 wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/17959#discussion_r116162790
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/DataSourceScanExec.scala
---
@@ -519,8 +519,18 @@ case class FileSourceScanExec(
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17956
This PR description is misleading. This PR is actually a bug fix, I think
---
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 p
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/16985#discussion_r116162386
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/BucketedReadSuite.scala ---
@@ -315,8 +317,14 @@ abstract class BucketedReadSuite extends
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17948
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 and wishes so, or if the featur
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17948#discussion_r116161956
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1003,18 +1003,32 @@ class Analyzer(
*/
ob
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17948#discussion_r116161811
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1003,18 +1003,32 @@ class Analyzer(
*/
ob
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17956#discussion_r116161668
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -126,16 +125,11 @@ class JacksonParser(
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17956#discussion_r116161266
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -126,16 +125,11 @@ class JacksonParser(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17644
**[Test build #76852 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76852/testReport)**
for PR 17644 at commit
[`239beee`](https://github.com/apache/spark/commit/23
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17901#discussion_r116160628
--- Diff: sql/core/src/main/scala/org/apache/spark/sql/functions.scala ---
@@ -2683,13 +2683,12 @@ object functions {
def unix_timestamp(s: Column
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17956#discussion_r116160516
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -126,16 +125,11 @@ class JacksonParser(
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17956#discussion_r116160371
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -126,16 +125,11 @@ class JacksonParser(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17956#discussion_r116160121
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -126,16 +125,11 @@ class JacksonParser(
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17948#discussion_r116159902
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1003,18 +1003,32 @@ class Analyzer(
*/
ob
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17956
**[Test build #76851 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76851/testReport)**
for PR 17956 at commit
[`e858789`](https://github.com/apache/spark/commit/e8
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17956#discussion_r116159720
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -126,16 +125,11 @@ class JacksonParser(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17956#discussion_r116159435
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/json/JacksonParser.scala
---
@@ -126,16 +125,11 @@ class JacksonParser(
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17956#discussion_r116159393
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1988,4 +1988,47 @@ class JsonSuite extends Qu
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76850 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76850/testReport)**
for PR 17711 at commit
[`de89791`](https://github.com/apache/spark/commit/de
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17948#discussion_r116159338
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1003,18 +1003,32 @@ class Analyzer(
*/
ob
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17956#discussion_r116159278
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1988,4 +1988,47 @@ class JsonSuite extends Que
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17956#discussion_r116159220
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1988,4 +1988,47 @@ class JsonSuite extends Que
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17711#discussion_r116159196
--- Diff: sql/core/src/test/resources/sql-tests/inputs/operators.sql ---
@@ -32,3 +32,11 @@ select 1 - 2;
select 2 * 5;
select 5 % 3;
select pm
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/17711
@rxin ok, thank for the suggestion!
---
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 ScrapCodes commented on a diff in the pull request:
https://github.com/apache/spark/pull/17308#discussion_r116158469
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaSink.scala
---
@@ -30,14 +30,19 @@ private[kafka010] class KafkaSink
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17945
---
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 cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16985#discussion_r116158389
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/ReorderJoinPredicates.scala
---
@@ -0,0 +1,93 @@
+/*
+ * Licensed to the A
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17958
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76844/
Test FAILed.
---
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/17945
merged to master/2.2
---
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
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17958
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 rxin commented on the issue:
https://github.com/apache/spark/pull/17711
I feel both are pretty complicated. Can we just do something similar to
CombineUnion:
```
/**
* Combines all adjacent [[Union]] operators into a single [[Union]].
*/
object Co
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17958
**[Test build #76844 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76844/testReport)**
for PR 17958 at commit
[`e10101e`](https://github.com/apache/spark/commit/e
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16985#discussion_r116158180
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/joins/ReorderJoinPredicates.scala
---
@@ -0,0 +1,93 @@
+/*
+ * Licensed to the A
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/17945
AppVeyor was failing because of the `SparkListenerBus` issue.
---
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 doe
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/16985#discussion_r116158076
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/BucketedReadSuite.scala ---
@@ -315,8 +317,14 @@ abstract class BucketedReadSuite extends Q
Github user tejasapatil commented on a diff in the pull request:
https://github.com/apache/spark/pull/17644#discussion_r116157920
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -871,6 +886,23 @@ private[hive] object HiveClientImpl {
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17956
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17956
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76846/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17956
**[Test build #76846 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76846/testReport)**
for PR 17956 at commit
[`aa7c658`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17958
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17958
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76845/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17958
**[Test build #76845 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76845/testReport)**
for PR 17958 at commit
[`1e36134`](https://github.com/apache/spark/commit/1
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17953#discussion_r116157223
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/parser/AstBuilder.scala
---
@@ -1504,6 +1504,7 @@ class AstBuilder extends SqlBaseBa
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17959
merged to master/2.2, please send a follow-up PR to address @gatorsmile 's
comments, thanks!
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17711#discussion_r116156736
--- Diff: sql/core/src/test/resources/sql-tests/inputs/operators.sql ---
@@ -32,3 +32,11 @@ select 1 - 2;
select 2 * 5;
select 5 % 3;
select pm
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17711
@maropu The solution using `tailrec` looks more straightforward. Could you
submit the PR based on that? Thanks!
---
If your project is set up for it, you can reply to this email and have your
re
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17711#discussion_r116156556
--- Diff: sql/core/src/test/resources/sql-tests/inputs/operators.sql ---
@@ -32,3 +32,11 @@ select 1 - 2;
select 2 * 5;
select 5 % 3;
selec
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17959
---
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 gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17959
How about `HiveTableScanExec`?
---
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 gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17959#discussion_r116156087
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/DataSourceScanExec.scala
---
@@ -519,8 +519,18 @@ case class FileSourceScanExec(
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17948#discussion_r116155996
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1003,18 +1003,32 @@ class Analyzer(
*/
ob
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/14582
We could ask it to mailing-list if you strongly feel about this. For
example, `from_json` function was also asked too to mailing list before getting
merged.
I think we should not add al
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17948#discussion_r116155780
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1003,18 +1003,32 @@ class Analyzer(
*/
ob
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/17948
LGTM 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 your project does not have this feature
enabled and wishes so, or
Github user ghoto commented on a diff in the pull request:
https://github.com/apache/spark/pull/17940#discussion_r116155652
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/linalg/Matrices.scala
---
@@ -992,7 +992,20 @@ object Matrices {
new DenseMatrix(dm.rows, d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17959
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76843/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17959
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17959
**[Test build #76843 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76843/testReport)**
for PR 17959 at commit
[`9ec86ec`](https://github.com/apache/spark/commit/9
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17957
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17957
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76842/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17957
**[Test build #76842 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76842/testReport)**
for PR 17957 at commit
[`4032940`](https://github.com/apache/spark/commit/4
Github user tejasapatil commented on the issue:
https://github.com/apache/spark/pull/16985
@cloud-fan : I have made suggested change(s).
---
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 f
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17956
LGTM except for a minor comment.
---
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 viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/17956#discussion_r116154314
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/json/JsonSuite.scala
---
@@ -1988,4 +1988,47 @@ class JsonSuite extends QueryTe
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/17052
Yea, I just wanted to check if it is in progress in any way. Thanks for
your input.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as wel
Github user lalinsky commented on the issue:
https://github.com/apache/spark/pull/14582
You were asking for more interest in the feature, there was no way I could
answer that. :)
Regarding the change itself, the system can already auto cast integer to a
timestamp, but not a f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17711
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76840/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17711
**[Test build #76840 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76840/testReport)**
for PR 17711 at commit
[`089db30`](https://github.com/apache/spark/commit/0
Github user uncleGen commented on the issue:
https://github.com/apache/spark/pull/17052
@HyukjinKwon Sorry! Busy for this period of time. Let me resolve this
conflict.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If y
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17956
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/76841/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17956
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17956
**[Test build #76841 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76841/testReport)**
for PR 17956 at commit
[`660a284`](https://github.com/apache/spark/commit/6
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17935#discussion_r116153501
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/SubquerySuite.scala
---
@@ -868,6 +868,29 @@ class SubquerySuite extends QueryTest with
SharedSQLC
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17948
**[Test build #76849 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76849/testReport)**
for PR 17948 at commit
[`a809274`](https://github.com/apache/spark/commit/a8
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17960#discussion_r116153263
--- Diff: sql/core/src/test/resources/sql-tests/inputs/limit.sql ---
@@ -1,23 +1,27 @@
-- limit on various data types
-select * from testd
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17948#discussion_r116153034
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1003,18 +1003,31 @@ class Analyzer(
*/
ob
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17644#discussion_r116152814
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -871,6 +886,23 @@ private[hive] object HiveClientImpl {
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/17948#discussion_r116152643
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1003,18 +1003,31 @@ class Analyzer(
*/
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17959
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 and wishes so, or if the fea
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17948
**[Test build #76848 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76848/testReport)**
for PR 17948 at commit
[`0163656`](https://github.com/apache/spark/commit/01
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17960
**[Test build #76847 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76847/testReport)**
for PR 17960 at commit
[`b4a4b0a`](https://github.com/apache/spark/commit/b4
GitHub user gatorsmile opened a pull request:
https://github.com/apache/spark/pull/17960
[SPARK-20719] [SQL] Support LIMIT ALL
### What changes were proposed in this pull request?
`LIMIT ALL` is the same as omitting the `LIMIT` clause. It is supported by
both PrestgreSQL and Pre
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/17948#discussion_r116151791
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1003,18 +1003,30 @@ class Analyzer(
*/
ob
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17948
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 and wishes so, or if the fea
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17948#discussion_r116150714
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/Analyzer.scala
---
@@ -1003,18 +1003,30 @@ class Analyzer(
*/
Github user ConeyLiu commented on the issue:
https://github.com/apache/spark/pull/17936
The cluster test result. The `RDD.cartesian` is used in Spark mllib ALS
algorithm, and compared with the latest spark master branch.
Environments: Spark on Yarn with 9 executors(10 cores
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/16697
Yes. If inside, you are right - only the first will be logged !
---
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 n
Github user CodingCat commented on the issue:
https://github.com/apache/spark/pull/16697
you mean outside of
https://github.com/apache/spark/pull/16697/files#diff-ca0fe05a42fd5edcab8a1bdaa8e58db9R210?
---
If your project is set up for it, you can reply to this email and have your
rep
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/17942#discussion_r116148995
--- Diff: core/src/main/scala/org/apache/spark/scheduler/Task.scala ---
@@ -115,26 +115,33 @@ private[spark] abstract class Task[T](
case t: T
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/17942#discussion_r116148841
--- Diff: core/src/main/scala/org/apache/spark/util/taskListeners.scala ---
@@ -55,14 +55,16 @@ class TaskCompletionListenerException(
extends Runtime
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17956
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 and wishes so, or if the fea
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17906
thanks, merging to master/2.2/2.1/2.0!
---
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
en
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17906
---
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 wzhfy commented on the issue:
https://github.com/apache/spark/pull/17959
also cc @gatorsmile
---
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, o
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17958
**[Test build #76845 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/76845/testReport)**
for PR 17958 at commit
[`1e36134`](https://github.com/apache/spark/commit/1e
1 - 100 of 630 matches
Mail list logo