Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17556
**[Test build #3654 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3654/testReport)**
for PR 17556 at commit
[`9ca5750`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17077
**[Test build #75658 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75658/testReport)**
for PR 17077 at commit
[`7b93482`](https://github.com/apache/spark/commit/7b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17077
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
e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17077
**[Test build #75658 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75658/testReport)**
for PR 17077 at commit
[`7b93482`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17077
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75658/
Test FAILed.
---
Github user guoxiaolongzte commented on the issue:
https://github.com/apache/spark/pull/17580
@jerryshao
import org.apache.kafka.clients.producer.KafkaProducer;
import org.apache.kafka.clients.producer.ProducerConfig;
import org.apache.kafka.clients.producer.ProducerRe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17590
**[Test build #75656 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75656/testReport)**
for PR 17590 at commit
[`ed0dd86`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17590
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/17590
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75656/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17556
**[Test build #3654 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3654/testReport)**
for PR 17556 at commit
[`9ca5750`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17077
**[Test build #75659 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75659/testReport)**
for PR 17077 at commit
[`845ee87`](https://github.com/apache/spark/commit/84
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17077
**[Test build #75659 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75659/testReport)**
for PR 17077 at commit
[`845ee87`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17077
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
e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17077
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75659/
Test FAILed.
---
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17077#discussion_r110630404
--- Diff: python/pyspark/sql/readwriter.py ---
@@ -545,6 +545,57 @@ def partitionBy(self, *cols):
self._jwrite = self._jwrite.partitionBy(_t
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17077#discussion_r110632132
--- Diff: python/pyspark/sql/tests.py ---
@@ -2167,6 +2167,61 @@ def test_BinaryType_serialization(self):
df = self.spark.createDataFrame(da
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/17077#discussion_r110634385
--- Diff: python/pyspark/sql/tests.py ---
@@ -2167,6 +2167,61 @@ def test_BinaryType_serialization(self):
df = self.spark.createDataFrame(da
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17586
**[Test build #75655 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75655/testReport)**
for PR 17586 at commit
[`0daa041`](https://github.com/apache/spark/commit/0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17586
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/17586
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75655/
Test PASSed.
---
GitHub user bogdanrdc opened a pull request:
https://github.com/apache/spark/pull/17591
[SPARK-20280][CORE] FileStatusCache Weigher integer overflow
## What changes were proposed in this pull request?
Weigher.weigh needs to return Int but it is possible for an
Array[FileSta
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17591
**[Test build #75660 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75660/testReport)**
for PR 17591 at commit
[`f4ae52a`](https://github.com/apache/spark/commit/f4
GitHub user bogdanrdc opened a pull request:
https://github.com/apache/spark/pull/17592
[SPARK-20243][TESTS] DebugFilesystem.assertNoOpenStreams thread race
## What changes were proposed in this pull request?
Synchronize access to openStreams map.
## How was this pa
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17592
**[Test build #75661 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75661/testReport)**
for PR 17592 at commit
[`19ca5a1`](https://github.com/apache/spark/commit/19
GitHub user guoxiaolongzte opened a pull request:
https://github.com/apache/spark/pull/17593
[SPARK-20279][WEB-UI]In web ui,'Only showing 200' should be changed to
'only showing last 200'.
## What changes were proposed in this pull request?
In web ui,'Only showing 200' shou
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17591#discussion_r110638961
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileStatusCache.scala
---
@@ -94,13 +94,25 @@ private class SharedInMemoryCache
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17593
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 srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17591#discussion_r110639538
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileStatusCache.scala
---
@@ -94,13 +94,25 @@ private class SharedInMemoryCache
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17592#discussion_r110639779
--- Diff: core/src/test/scala/org/apache/spark/DebugFilesystem.scala ---
@@ -31,21 +30,29 @@ import org.apache.spark.internal.Logging
object DebugF
Github user bogdanrdc commented on a diff in the pull request:
https://github.com/apache/spark/pull/17591#discussion_r110639886
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileStatusCache.scala
---
@@ -94,13 +94,25 @@ private class SharedInMemoryCa
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17591#discussion_r110642316
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileStatusCache.scala
---
@@ -94,13 +94,25 @@ private class SharedInMemoryCache
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/17593
I'm not sure this is accurate, given how sort order can be changed? maybe
not, but it's at least ambiguous, and don't think it adds enough to change this.
---
If your project is set up for it, you c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17556
**[Test build #3655 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3655/testReport)**
for PR 17556 at commit
[`9ca5750`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17592
**[Test build #75662 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75662/testReport)**
for PR 17592 at commit
[`d486d60`](https://github.com/apache/spark/commit/d4
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/17592
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 fe
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17591
**[Test build #75663 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75663/testReport)**
for PR 17591 at commit
[`ef8e9e9`](https://github.com/apache/spark/commit/ef
Github user bogdanrdc commented on a diff in the pull request:
https://github.com/apache/spark/pull/17591#discussion_r110646607
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileStatusCache.scala
---
@@ -94,13 +94,25 @@ private class SharedInMemoryCa
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17556
**[Test build #3655 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/3655/testReport)**
for PR 17556 at commit
[`9ca5750`](https://github.com/apache/spark/commit/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17330
**[Test build #75654 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75654/testReport)**
for PR 17330 at commit
[`22db44a`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17330
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75654/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17330
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 tgravescs commented on the issue:
https://github.com/apache/spark/pull/17582
Sorry but I'm confused by the explanation in the description. I didn't
completely follow what problems you are seeing that aren't intended and I don't
understand how you are proposing to fix. Ca
Github user Syrux commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110661386
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan extends
Github user Syrux commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110661717
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan extends
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110662125
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan extends
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110662332
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan extends
Github user Syrux commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110662506
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan extends
Github user Syrux commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110662589
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala ---
@@ -360,6 +360,55 @@ class PrefixSpanSuite extends SparkFunSuite with
MLl
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17527
**[Test build #75657 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75657/testReport)**
for PR 17527 at commit
[`2ac5843`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17527
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/17527
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75657/
Test PASSed.
---
Github user Syrux commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110664282
--- Diff:
mllib/src/test/scala/org/apache/spark/mllib/fpm/PrefixSpanSuite.scala ---
@@ -360,6 +360,55 @@ class PrefixSpanSuite extends SparkFunSuite with
MLl
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17591
**[Test build #75660 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75660/testReport)**
for PR 17591 at commit
[`f4ae52a`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17591
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/17591
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75660/
Test PASSed.
---
Github user Syrux commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110667171
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan extends
Github user Syrux commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110669561
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan extends
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/17587
@kiszk The plan may not be very helpful, as the only difference in the plan
is `MapObjects.lambdaFunction`, before it was just `LambdaVariable`, but now
it's `Cast(LambdaVariable, xxx)`
---
If y
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17330#discussion_r110671091
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/subquery.scala
---
@@ -236,6 +244,12 @@ case class ScalarSubquery(
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110671100
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17330#discussion_r110671334
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/subquery.scala
---
@@ -59,6 +58,13 @@ abstract class SubqueryExpression(
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/17330#discussion_r110671813
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/subquery.scala
---
@@ -59,6 +58,13 @@ abstract class SubqueryExpression(
Github user Syrux commented on a diff in the pull request:
https://github.com/apache/spark/pull/17575#discussion_r110671916
--- Diff: mllib/src/main/scala/org/apache/spark/mllib/fpm/PrefixSpan.scala
---
@@ -232,6 +200,68 @@ class PrefixSpan private (
object PrefixSpan extends
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17587
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/17587#discussion_r110676717
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/ScalaReflection.scala
---
@@ -270,12 +270,13 @@ object ScalaReflection extends ScalaRefl
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17592
**[Test build #75661 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75661/testReport)**
for PR 17592 at commit
[`19ca5a1`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17592
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/17592
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75661/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17591
**[Test build #75663 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75663/testReport)**
for PR 17591 at commit
[`ef8e9e9`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17591
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/17591
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75663/
Test PASSed.
---
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/17491
cc @cloud-fan
---
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
Github user BenFradet commented on a diff in the pull request:
https://github.com/apache/spark/pull/17308#discussion_r110685760
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/CachedKafkaProducer.scala
---
@@ -0,0 +1,70 @@
+/*
+ * Licensed to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17592
**[Test build #75662 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75662/testReport)**
for PR 17592 at commit
[`d486d60`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17592
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75662/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17592
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 hvanhovell commented on the issue:
https://github.com/apache/spark/pull/17592
Merging to master. 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 w
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17592
---
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 hvanhovell commented on the issue:
https://github.com/apache/spark/pull/17566
LGTM - merging to master. 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
enable
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/17593
I agree with @srowen we left it that way since sorting can change
---
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 do
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17591#discussion_r110692150
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/FileIndexSuite.scala
---
@@ -220,6 +221,32 @@ class FileIndexSuite extends
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17566
---
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 hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17591#discussion_r110692271
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileStatusCache.scala
---
@@ -94,27 +94,46 @@ private class SharedInMemoryC
Github user hvanhovell commented on a diff in the pull request:
https://github.com/apache/spark/pull/17591#discussion_r110692833
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/FileStatusCache.scala
---
@@ -94,27 +94,46 @@ private class SharedInMemoryC
Github user zero323 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17077#discussion_r110692936
--- Diff: python/pyspark/sql/tests.py ---
@@ -2167,6 +2167,61 @@ def test_BinaryType_serialization(self):
df = self.spark.createDataFrame(data,
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/17330#discussion_r110693397
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/subquery.scala
---
@@ -59,6 +58,13 @@ abstract class SubqueryExpression
Github user zero323 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17077#discussion_r110693499
--- Diff: python/pyspark/sql/tests.py ---
@@ -2167,6 +2167,61 @@ def test_BinaryType_serialization(self):
df = self.spark.createDataFrame(data,
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/17587
@cloud-fan Good to see what plan is generated in your comment. People
(especially I) will forgot what plan we generated in the future.
When I saw the comment in `upCastToExpectedType`, I understoo
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17330
**[Test build #75664 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75664/testReport)**
for PR 17330 at commit
[`f1e63c8`](https://github.com/apache/spark/commit/f1
Github user BenFradet commented on the issue:
https://github.com/apache/spark/pull/12823
ping @jkbradley if you could take a look, that'd be great.
If you have the time, there is also the #17431 segue.
---
If your project is set up for it, you can reply to this email and have
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17591
**[Test build #75665 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75665/testReport)**
for PR 17591 at commit
[`e02fc4a`](https://github.com/apache/spark/commit/e0
Github user hvanhovell commented on the issue:
https://github.com/apache/spark/pull/17591
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 fe
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/17585
---
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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/17077
**[Test build #75666 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75666/testReport)**
for PR 17077 at commit
[`481416d`](https://github.com/apache/spark/commit/48
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17077
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/75666/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17077
**[Test build #75666 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75666/testReport)**
for PR 17077 at commit
[`481416d`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/17077
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
e
Github user zero323 commented on a diff in the pull request:
https://github.com/apache/spark/pull/17077#discussion_r110699779
--- Diff: python/pyspark/sql/tests.py ---
@@ -2167,6 +2167,61 @@ def test_BinaryType_serialization(self):
df = self.spark.createDataFrame(data,
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/17364
@squito Is this ready to go in? Like I warned, I'm not going to add tests
for this, not on its own
---
If your project is set up for it, you can reply to this email and have your
reply appear
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/17077
**[Test build #75667 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/75667/testReport)**
for PR 17077 at commit
[`71c9e0f`](https://github.com/apache/spark/commit/71
101 - 200 of 422 matches
Mail list logo