Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22237#discussion_r223254210
--- Diff: docs/sql-programming-guide.md ---
@@ -1890,6 +1890,10 @@ working with timestamps in `pandas_udf`s to get the
best performance, see
#
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22635
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97100/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22635
**[Test build #97100 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97100/testReport)**
for PR 22635 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22635
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user MaxGekk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22237#discussion_r223263164
--- Diff: docs/sql-programming-guide.md ---
@@ -1890,6 +1890,10 @@ working with timestamps in `pandas_udf`s to get the
best performance, see
#
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22669
**[Test build #97103 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97103/testReport)**
for PR 22669 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22594
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/22594
**[Test build #97104 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97104/testReport)**
for PR 22594 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22594
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/22615
**[Test build #97105 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97105/testReport)**
for PR 22615 at commit
Github user adrian555 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22455#discussion_r223257553
--- Diff: R/pkg/R/DataFrame.R ---
@@ -246,30 +248,38 @@ setMethod("showDF",
#' @note show(SparkDataFrame) since 1.4.0
setMethod("show",
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22667
**[Test build #97092 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97092/testReport)**
for PR 22667 at commit
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22670
cc @gatorsmile
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user shivusondur opened a pull request:
https://github.com/apache/spark/pull/22669
[SPARK-25677] [Spark Compression] spark.io.compression.codec =
org.apache.spark.io.ZstdCompressionCodec throwing IllegalArgumentException
Exception
## What changes were proposed in this pull
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22669
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22669
Please update title to:
```
[SPARK-25677][DOC] ...
```
---
-
To unsubscribe, e-mail:
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/22669
cc @HyukjinKwon
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22667
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/22670
**[Test build #97102 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97102/testReport)**
for PR 22670 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22667
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97092/
Test PASSed.
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22667
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22665
---
-
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/22667
Merged to master and branch-2.4.
Thanks @srowen and @felixcheung.
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/13326
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/13379
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/22670
**[Test build #97102 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97102/testReport)**
for PR 22670 at commit
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22669
@felixcheung I searched in all the code base, only this place i found
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22630
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97101/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22594
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22630
**[Test build #97101 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97101/testReport)**
for PR 22630 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22630
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/22594
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97095/
Test FAILed.
---
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22669
ok to test
---
-
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/22594
**[Test build #97095 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97095/testReport)**
for PR 22594 at commit
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22670
[SPARK-25631][SPARK-25632][SQL][TEST] Improve the test runtime of
KafkaRDDSuite
## What changes were proposed in this pull request?
Set a reasonable poll timeout thats used while consuming
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22669
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/22669
**[Test build #97103 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97103/testReport)**
for PR 22669 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22669
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97103/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22669
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/22670
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/22669
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/22670
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/22670
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97102/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22670
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/22594
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22635
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22630
**[Test build #97101 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97101/testReport)**
for PR 22630 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22630
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/22630
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21732
Since we have to handle some special cases, can you list them in the PR
description?
For `Product`, normally it should be a struct type column, the special case
I can think of is: 1) at
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22594
**[Test build #97096 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97096/testReport)**
for PR 22594 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22594
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97096/
Test PASSed.
---
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22594
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/22635
Merged to master and branch-2.4.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/22630
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21996
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/22482
**[Test build #97107 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97107/testReport)**
for PR 22482 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22631#discussion_r223314632
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CastSuite.scala
---
@@ -110,7 +112,7 @@ class CastSuite extends
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22655
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22631#discussion_r223292455
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CastSuite.scala
---
@@ -110,7 +112,7 @@ class CastSuite extends
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22630#discussion_r223318798
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -362,8 +362,15 @@ trait CodegenSupport extends
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22669
Merged to master and branch-2.4.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user shivusondur commented on the issue:
https://github.com/apache/spark/pull/22668
@felixcheung If the user inputs **rowcount** in **Show** tab such that all
the rows comes in one page,
From here on user can't see the pagination and he will not able to inputs
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22655
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22630
**[Test build #97106 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97106/testReport)**
for PR 22630 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22630
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/22630
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22631#discussion_r223286771
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CastSuite.scala
---
@@ -110,7 +112,7 @@ class CastSuite extends
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22594
**[Test build #97104 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97104/testReport)**
for PR 22594 at commit
Github user ScrapCodes closed the pull request at:
https://github.com/apache/spark/pull/22564
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user ScrapCodes commented on the issue:
https://github.com/apache/spark/pull/22564
Looks like this is working without making a release. It is not clear what
change could have fixed the problem. Closing the PR for now.
---
Github user ScrapCodes commented on the issue:
https://github.com/apache/spark/pull/22564
I am sorry for the trouble, @liyinan926 and @srowen.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22630#discussion_r223319859
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -345,6 +345,61 @@ trait CodegenSupport extends
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22630#discussion_r223319524
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -345,6 +345,61 @@ trait CodegenSupport extends
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22655
I am getting this in. This is an ongoing effort and it just documents them
internally for now.
---
-
To unsubscribe,
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22669
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22631#discussion_r223285813
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CastSuite.scala
---
@@ -110,7 +112,7 @@ class CastSuite extends
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22630#discussion_r223315367
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -360,6 +360,10 @@ trait CodegenSupport extends
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20999
kindly ping @cloud-fan @dongjoon-hyun @gatorsmile @viirya
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22594
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/22594
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97104/
Test PASSed.
---
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/22630#discussion_r223277348
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/WholeStageCodegenExec.scala
---
@@ -360,6 +360,10 @@ trait CodegenSupport extends
GitHub user dilipbiswal opened a pull request:
https://github.com/apache/spark/pull/22671
[SPARK-25615][SQL][TEST] Improve the test runtime of KafkaSinkSuite:
streaming write to non-existing topic
## What changes were proposed in this pull request?
Specify `kafka.max.block.ms`
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22029
@cloud-fan @dongjoon-hyun @gatorsmile anymore comments on this?
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22615
Build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22615
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97105/
Test FAILed.
---
GitHub user srowen opened a pull request:
https://github.com/apache/spark/pull/22672
[SPARK-25605][TESTS] Alternate take. Run cast string to timestamp tests for
a subset of timezones
## What changes were proposed in this pull request?
Try testing timezones in parallel
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r223354399
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/TokenUtil.scala
---
@@ -0,0 +1,116 @@
+/*
+ * Licensed to the
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r223354478
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/TokenUtil.scala
---
@@ -0,0 +1,116 @@
+/*
+ * Licensed to the
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22657
I agree. We don't test `commons-lang3` library here.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22671
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/22671
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/22598#discussion_r223352683
--- Diff:
core/src/test/scala/org/apache/spark/deploy/security/HadoopDelegationTokenManagerSuite.scala
---
@@ -111,6 +113,17 @@ class
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22631#discussion_r223354745
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CastSuite.scala
---
@@ -110,7 +112,7 @@ class CastSuite extends
Github user misutoth commented on the issue:
https://github.com/apache/spark/pull/22532
Many thanks for the feedback. I will list the test scenarios that I had in
mind and collected while I implemented this item.
And sorry about the failure, seems like I did not rerun all the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22029
**[Test build #97110 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97110/testReport)**
for PR 22029 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22630
**[Test build #97111 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97111/testReport)**
for PR 22630 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22598
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/97112/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22672
**[Test build #97113 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97113/testReport)**
for PR 22672 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22598
**[Test build #97114 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/97114/testReport)**
for PR 22598 at commit
1 - 100 of 566 matches
Mail list logo