Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21152
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89877/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21152
**[Test build #89877 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89877/testReport)**
for PR 21152 at commit
[`3eee5c9`](https://github.com/apache/spark/commit/3
Github user mn-mikke commented on a diff in the pull request:
https://github.com/apache/spark/pull/21028#discussion_r184379580
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -288,6 +288,114 @@ case class ArrayConta
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21147
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21147
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89884/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21147
**[Test build #89884 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89884/testReport)**
for PR 21147 at commit
[`9672f92`](https://github.com/apache/spark/commit/9
Github user mshtelma commented on a diff in the pull request:
https://github.com/apache/spark/pull/21147#discussion_r184376159
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -392,13 +392,13 @@ case cl
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21167
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21167
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20146
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2693/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20146
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21166
Can you please check again with latest master code, I doubt the issue is
not valid any more in the latest code.
---
-
To unsub
GitHub user WzRaCai opened a pull request:
https://github.com/apache/spark/pull/21167
[SPARK-24100][PYSPARK][DStreams]Add the CompressionCodec to the
saveAsTextFiles interface.
## What changes were proposed in this pull request?
Add the CompressionCodec to the saveAsTextFil
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20146
**[Test build #89886 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89886/testReport)**
for PR 20146 at commit
[`a6551b0`](https://github.com/apache/spark/commit/a6
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/20930
No wonder I can't understand the issue for a long time since I've thought
it happened on Spark2.3 . And now it makes sense. Thanks @jiangxb1987
---
---
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/20930
> Have you applied this patch: #17955 ?
No, this happened on Spark 2.1. Thanks xingbo & wenchen, I'll port back
this patch to our internal Spark 2.1.
> That PR seems to be addr
Github user efimpoberezkin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20936#discussion_r184361843
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/EpochCoordinator.scala
---
@@ -137,30 +137,65 @@ private[conti
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20936
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89876/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21156
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21156
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89875/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20936
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20936
**[Test build #89876 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89876/testReport)**
for PR 20936 at commit
[`b1b9985`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21156
**[Test build #89875 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89875/testReport)**
for PR 21156 at commit
[`a59c94f`](https://github.com/apache/spark/commit/a
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21106
I think `UnsafeProjectionCreator` is only useful for tests, because in
production we should always try codegen first, and fallback to interpretation
if codegen fails. There is no need to disable t
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21021
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21021
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89873/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21021
**[Test build #89873 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89873/testReport)**
for PR 21021 at commit
[`f2798f9`](https://github.com/apache/spark/commit/f
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21136
**[Test build #89885 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89885/testReport)**
for PR 21136 at commit
[`fed913d`](https://github.com/apache/spark/commit/fe
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20140
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21028#discussion_r184351841
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -288,6 +288,114 @@ case class ArrayCont
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21157
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89883/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21157
**[Test build #89883 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89883/testReport)**
for PR 21157 at commit
[`c67ce29`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21157
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21147
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21147
somehow I thought it has passed tests and I has merged it to master...
Anyway this is a pretty safe change and I don't think it will break any tests.
Let's see the test result later.
---
--
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20146
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89872/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20146
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20146
**[Test build #89872 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89872/testReport)**
for PR 20146 at commit
[`c33cff6`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21147
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2692/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21147
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21147
**[Test build #89884 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89884/testReport)**
for PR 21147 at commit
[`9672f92`](https://github.com/apache/spark/commit/96
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21166
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21166
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user juliuszsompolski commented on a diff in the pull request:
https://github.com/apache/spark/pull/21133#discussion_r184343803
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/ApproximatePercentileQuerySuite.scala
---
@@ -279,4 +282,10 @@ class ApproximatePercentileQ
Github user juliuszsompolski commented on a diff in the pull request:
https://github.com/apache/spark/pull/21133#discussion_r184347998
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/aggregate/ApproximatePercentile.scala
---
@@ -238,12 +238,6 @@ obj
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/21147
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
GitHub user sadhen opened a pull request:
https://github.com/apache/spark/pull/21166
[SPARK-11334][CORE] clear idle executors in executorIdToTaskIds keySet
## What changes were proposed in this pull request?
quote from #11205
> Executors may never be idle. Currentl
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21157
**[Test build #89883 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89883/testReport)**
for PR 21157 at commit
[`c67ce29`](https://github.com/apache/spark/commit/c6
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21119#discussion_r184342231
--- Diff: python/pyspark/ml/clustering.py ---
@@ -1156,6 +1156,201 @@ def getKeepLastCheckpoint(self):
return self.getOrDefault(self.keepLa
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21119#discussion_r184346287
--- Diff: python/pyspark/ml/clustering.py ---
@@ -1156,6 +1156,201 @@ def getKeepLastCheckpoint(self):
return self.getOrDefault(self.keepLa
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21119#discussion_r184343934
--- Diff: python/pyspark/ml/clustering.py ---
@@ -1156,6 +1156,201 @@ def getKeepLastCheckpoint(self):
return self.getOrDefault(self.keepLa
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21119#discussion_r184344777
--- Diff: python/pyspark/ml/clustering.py ---
@@ -1156,6 +1156,201 @@ def getKeepLastCheckpoint(self):
return self.getOrDefault(self.keepLa
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21119#discussion_r184344901
--- Diff: python/pyspark/ml/clustering.py ---
@@ -1156,6 +1156,201 @@ def getKeepLastCheckpoint(self):
return self.getOrDefault(self.keepLa
Github user WeichenXu123 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21119#discussion_r184345688
--- Diff: python/pyspark/ml/clustering.py ---
@@ -1156,6 +1156,201 @@ def getKeepLastCheckpoint(self):
return self.getOrDefault(self.keepLa
Github user superbobry commented on the issue:
https://github.com/apache/spark/pull/21157
Yes, we can backport some of the cloudpickle code to make the patch less
fragile. This would be a nontrivial change in an already complex code, but I'd
be happy to sketch this if there's a consen
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21106
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21106
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution/2691/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21106
**[Test build #89882 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89882/testReport)**
for PR 21106 at commit
[`68213fb`](https://github.com/apache/spark/commit/68
Github user efimpoberezkin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21063#discussion_r184342675
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/ProgressReporter.scala
---
@@ -111,7 +112,12 @@ trait ProgressReporter ex
Github user sadhen commented on the issue:
https://github.com/apache/spark/pull/11205
@jerryshao I think the 2nd bullet has not been fixed in SPARK-13054.
I use spark 2.1.1, and I still find that finished tasks remain in
`private val executorIdToTaskIds = new mutable.HashM
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21136
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21136
**[Test build #89880 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89880/testReport)**
for PR 21136 at commit
[`0f3bc0b`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21136
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89880/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21157
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89878/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20946
**[Test build #89881 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89881/testReport)**
for PR 20946 at commit
[`b35acf5`](https://github.com/apache/spark/commit/b3
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21157
**[Test build #89878 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89878/testReport)**
for PR 21157 at commit
[`67c4f67`](https://github.com/apache/spark/commit/6
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21136
**[Test build #89880 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89880/testReport)**
for PR 21136 at commit
[`0f3bc0b`](https://github.com/apache/spark/commit/0f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21157
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/21136
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/20946
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21157
I don't like the hack too but the complete removal just basically means we
are going to drop namedtuple supports in RDD without, for example, any
deprecation warnings. Spark is being super conse
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21157
**[Test build #89878 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89878/testReport)**
for PR 21157 at commit
[`67c4f67`](https://github.com/apache/spark/commit/67
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20959
**[Test build #89879 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89879/testReport)**
for PR 20959 at commit
[`d4d9d65`](https://github.com/apache/spark/commit/d4
Github user superbobry commented on the issue:
https://github.com/apache/spark/pull/21157
> Does the test even pass?
The tests should pass module the tests specifically checking the behaviour
being removed. I think the failing RDD test is in this group as well.
> Why
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/21136
okey to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spa
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/20946
okey to test
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: re
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21165
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21152
**[Test build #89877 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89877/testReport)**
for PR 21152 at commit
[`3eee5c9`](https://github.com/apache/spark/commit/3e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21165
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21152
Thanks fore reviewing. I have addressed review comments.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
F
Github user advancedxy commented on the issue:
https://github.com/apache/spark/pull/21165
cc @squito @jiangxb1987
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: r
GitHub user advancedxy opened a pull request:
https://github.com/apache/spark/pull/21165
Spark 20087: Attach accumulators / metrics to 'TaskKilled' end reason
## What changes were proposed in this pull request?
The ultimate goal is for listeners to onTaskEnd to receive metrics wh
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/spark/pull/21152#discussion_r184322699
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/continuous/ContinuousSuite.scala
---
@@ -66,157 +66,115 @@ class ContinuousSuite exten
Github user HeartSaVioR commented on a diff in the pull request:
https://github.com/apache/spark/pull/21152#discussion_r184321762
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/continuous/ContinuousSuite.scala
---
@@ -66,157 +66,115 @@ class ContinuousSuite exten
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21164
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21164
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user liutang123 commented on the issue:
https://github.com/apache/spark/pull/21164
Bug Reappearance:
1. Add Thread.sleep(1000 * 600) before assign for _exception.
2. structure a python script witch will throw exception like follow:
test.py
```import sys
for l
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/21152#discussion_r184316769
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/continuous/ContinuousSuite.scala
---
@@ -66,157 +66,115 @@ class ContinuousSuite extends
GitHub user liutang123 opened a pull request:
https://github.com/apache/spark/pull/21164
[SPARK-24098][SQL] ScriptTransformationExec should wait process exiting
before output iterator finish
When feed thread doesn't set its _exception variable and the progress
doesn't exit complete
Github user jerryshao commented on a diff in the pull request:
https://github.com/apache/spark/pull/21152#discussion_r184315510
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/continuous/ContinuousSuite.scala
---
@@ -66,157 +66,115 @@ class ContinuousSuite extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21163
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/89874/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21163
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21163
**[Test build #89874 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89874/testReport)**
for PR 21163 at commit
[`cd5ea88`](https://github.com/apache/spark/commit/c
Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21028#discussion_r184310311
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -288,6 +288,114 @@ case class ArrayContain
Github user wzhfy commented on the issue:
https://github.com/apache/spark/pull/21147
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.
Github user wzhfy commented on a diff in the pull request:
https://github.com/apache/spark/pull/21147#discussion_r184308017
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/plans/logical/statsEstimation/FilterEstimation.scala
---
@@ -392,13 +392,13 @@ case class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21028#discussion_r184305294
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -288,6 +288,114 @@ case class ArrayCont
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20936
**[Test build #89876 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/89876/testReport)**
for PR 20936 at commit
[`b1b9985`](https://github.com/apache/spark/commit/b1
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/20930
Have you applied this patch: https://github.com/apache/spark/pull/17955 ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@s
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/21021
Based on the recent trend, I will support codegen to exploit the
opportunity of WholeStageCodegen.
---
-
To unsubscribe, e-mail: r
301 - 400 of 431 matches
Mail list logo
