Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23215
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99770/
Test FAILed.
---
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/23160
cc @tgravescs @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23215
**[Test build #99770 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99770/testReport)**
for PR 23215 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23248
**[Test build #99773 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99773/testReport)**
for PR 23248 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23248
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23248
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user shahidki31 commented on a diff in the pull request:
https://github.com/apache/spark/pull/23241#discussion_r239496266
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/ReplayListenerBus.scala ---
@@ -118,8 +118,6 @@ private[spark] class ReplayListenerBus extends
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23248
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user shahidki31 commented on a diff in the pull request:
https://github.com/apache/spark/pull/23241#discussion_r239494478
--- Diff: core/src/main/scala/org/apache/spark/io/CompressionCodec.scala ---
@@ -197,4 +201,8 @@ class ZStdCompressionCodec(conf: SparkConf) extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22612
**[Test build #99772 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99772/testReport)**
for PR 22612 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23195
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23195
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99771/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23195
**[Test build #99771 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99771/testReport)**
for PR 23195 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23195
**[Test build #99771 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99771/testReport)**
for PR 23195 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22683#discussion_r239482033
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/NullExpressionsSuite.scala
---
@@ -160,7 +160,7 @@ class
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22683#discussion_r239480795
--- Diff:
core/src/main/scala/org/apache/spark/serializer/KryoSerializer.scala ---
@@ -62,14 +62,14 @@ class KryoSerializer(conf: SparkConf)
if
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22683#discussion_r239481508
--- Diff: docs/sql-programming-guide.md ---
@@ -4,10 +4,15 @@ displayTitle: Spark SQL, DataFrames and Datasets Guide
title: Spark SQL and DataFrames
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/23195#discussion_r239480501
--- Diff: docs/structured-streaming-kafka-integration.md ---
@@ -624,3 +624,199 @@ For experimenting on `spark-shell`, you can also use
`--packages`
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22683
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22683
**[Test build #99763 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99763/testReport)**
for PR 22683 at commit
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23241#discussion_r239476570
--- Diff: core/src/main/scala/org/apache/spark/io/CompressionCodec.scala ---
@@ -197,4 +201,8 @@ class ZStdCompressionCodec(conf: SparkConf) extends
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/23241#discussion_r239476672
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/ReplayListenerBus.scala ---
@@ -118,8 +118,6 @@ private[spark] class ReplayListenerBus extends
Github user Ngone51 commented on the issue:
https://github.com/apache/spark/pull/23223
> it would be interesting to test it further to see if it does.
@tgravescs Yeah, I have the same thought. I'd like to try it, but I can
not guarantee that I can achieve it... Because I
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22759#discussion_r239474962
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/test/DataFrameReaderWriterSuite.scala
---
@@ -542,6 +551,35 @@ class DataFrameReaderWriterSuite
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22759#discussion_r239475332
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/test/DataFrameReaderWriterSuite.scala
---
@@ -542,6 +551,35 @@ class DataFrameReaderWriterSuite
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22759#discussion_r239475203
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/test/DataFrameReaderWriterSuite.scala
---
@@ -542,6 +551,35 @@ class DataFrameReaderWriterSuite
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23248
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23248
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99767/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23248
**[Test build #99767 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99767/testReport)**
for PR 23248 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23208#discussion_r239469368
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/SupportsBatchWrite.java
---
@@ -25,14 +25,14 @@
import
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/23223
if you aren't seeing actual issues with this I guess it would be
interesting to test it further to see if it does. I can see spark blacklisting
when it shouldn't for exit codes like you mention
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23215
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23215
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23215
**[Test build #99770 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99770/testReport)**
for PR 23215 at commit
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/23215#discussion_r239460682
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/PartitioningUtils.scala
---
@@ -272,9 +279,14 @@ object PartitioningUtils {
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22514
@cloud-fan I've updated the PR description. Thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23215
**[Test build #99769 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99769/testReport)**
for PR 23215 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23215
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23215
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/23208#discussion_r239454594
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/SupportsBatchWrite.java
---
@@ -25,14 +25,14 @@
import
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23215#discussion_r239453312
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -1396,6 +1396,16 @@ object SQLConf {
.booleanConf
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23215#discussion_r239453026
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/FileIndexSuite.scala
---
@@ -95,6 +95,31 @@ class FileIndexSuite extends
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23213
these 3 combinations LGTM.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23244#discussion_r239451274
--- Diff:
core/src/main/java/org/apache/spark/unsafe/map/BytesToBytesMap.java ---
@@ -209,23 +205,14 @@ public BytesToBytesMap(
Github user kiszk closed the pull request at:
https://github.com/apache/spark/pull/21777
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23248
Thanks, @cloud-fan. I will take a look within tomorrow - don't block by me.
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23245
**[Test build #99768 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99768/testReport)**
for PR 23245 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23248
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23248
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23248
**[Test build #99767 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99767/testReport)**
for PR 23248 at commit
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/23248#discussion_r239441136
--- Diff: python/pyspark/sql/tests/test_udf.py ---
@@ -23,7 +23,7 @@
from pyspark import SparkContext
from pyspark.sql import
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23245
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23245
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23245
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23246
Ditto. Let's close these and focus on fixing actual bugs.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23247
@wangjiaochun, I think you better stop fixing trivial stuff in each PR.
Those stuff can be fixed when the codes around here is fixed, or let other
people fix it later.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23244
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99760/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23244
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23244
**[Test build #99760 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99760/testReport)**
for PR 23244 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23215
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23215
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23241
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23241
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99759/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23215
**[Test build #99766 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99766/testReport)**
for PR 23215 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23241
**[Test build #99759 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99759/testReport)**
for PR 23241 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23215
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99761/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23248
**[Test build #99765 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99765/testReport)**
for PR 23248 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23248
**[Test build #99765 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99765/testReport)**
for PR 23248 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/23248
cc @icexelloss @HyukjinKwon @ueshin @viirya @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23248
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99765/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23248
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23215
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23248
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23215
**[Test build #99761 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99761/testReport)**
for PR 23215 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23248
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23248#discussion_r239430315
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/ArrowEvalPythonExec.scala
---
@@ -60,8 +60,12 @@ private class
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23248#discussion_r239430084
--- Diff: python/pyspark/sql/tests/test_udf.py ---
@@ -23,7 +23,7 @@
from pyspark import SparkContext
from pyspark.sql import
GitHub user cloud-fan opened a pull request:
https://github.com/apache/spark/pull/23248
[SPARK-26293][SQL] Cast exception when having python udf in subquery
## What changes were proposed in this pull request?
This is a regression introduced by
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23213
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99762/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23213
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23245
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/99764/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23245
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23245
**[Test build #99764 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99764/testReport)**
for PR 23245 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23213
**[Test build #99762 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99762/testReport)**
for PR 23213 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23247
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gengliangwang commented on a diff in the pull request:
https://github.com/apache/spark/pull/23215#discussion_r239423651
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/PartitioningUtils.scala
---
@@ -272,9 +279,13 @@ object
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23247
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/23247
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user wangjiaochun opened a pull request:
https://github.com/apache/spark/pull/23247
[SPARK-26294][CORE]Delete Unnecessary If statement
## What changes were proposed in this pull request?
Delete unnecessary If statement, because it Impossible execution when
records
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22952
I'm now also playing with Hadoop glob relevant classes to check whether
final destination matches source path glob pattern or not.
* Looks like we can leverage `GlobPattern` but it is
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23246
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/23246
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/23246
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user wangjiaochun opened a pull request:
https://github.com/apache/spark/pull/23246
[SPARK-26292][CORE]Assert statement of currentPage may be not in right place
## What changes were proposed in this pull request?
The assert statement of currentPage is not in right
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23245
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23245
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23245
**[Test build #99764 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/99764/testReport)**
for PR 23245 at commit
GitHub user ueshin opened a pull request:
https://github.com/apache/spark/pull/23245
[SPARK-26060][SQL][FOLLOW-UP] Rename the config name.
## What changes were proposed in this pull request?
This is a follow-up of #23031 to rename the config name.
The `execution`
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/23245
cc @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user weixiuli commented on the issue:
https://github.com/apache/spark/pull/23243
cc @gatorsmile @HyukjinKwon @ptkool @cloud-fan Kindly review
---
-
To unsubscribe, e-mail:
401 - 500 of 550 matches
Mail list logo