Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/23039
> let's fix PR description and title for that.
> One PR should have one theme with a proper title.
@HyukjinKwon @dongjoon-hyun I have renamed this PR. Is new title fine for
you?
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23030
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/23030
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98859/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23030
**[Test build #98859 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98859/testReport)**
for PR 23030 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23041
**[Test build #98860 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98860/testReport)**
for PR 23041 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23041
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/23041
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/23041#discussion_r233734198
--- Diff:
common/network-common/src/test/java/org/apache/spark/network/RpcIntegrationSuite.java
---
@@ -371,23 +371,29 @@ private void
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/23041
cc @squito
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user zsxwing opened a pull request:
https://github.com/apache/spark/pull/23041
[SPARK-26069][TESTS]Fix flaky test:
RpcIntegrationSuite.sendRpcWithStreamFailures
## What changes were proposed in this pull request?
The test failure is because `assertErrorAndClosed`
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23030
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/23030
**[Test build #98859 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98859/testReport)**
for PR 23030 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23030
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user uzadude commented on the issue:
https://github.com/apache/spark/pull/16732
After running some more experiments I was able to reduce the runtime by
another 1.5x factor. So currently the
"toCoordinateMatrix().toIndexedRowMatrix()" is better by a bit only in the
extreme
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23029
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/23029
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98854/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23029
**[Test build #98854 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98854/testReport)**
for PR 23029 at commit
Github user caneGuy closed the pull request at:
https://github.com/apache/spark/pull/21526
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user KyleLi1985 commented on the issue:
https://github.com/apache/spark/pull/22893
> Thanks @KyleLi1985 this looks like a nice win in the end. Thanks for your
investigation.
@srowen @HyukjinKwon @mgaido91 Thanks for review. It is my pleasure.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23038
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/23031
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/23031
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/23038
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98850/
Test PASSed.
---
Github user advancedxy commented on the issue:
https://github.com/apache/spark/pull/23040
cc @ericl and @JoshRosen, this bug was introduced by
https://github.com/apache/spark/pull/14099/files
After loosing empty chunk check, the ChunkedByteBufferInputStream doesn't
handle
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23038
**[Test build #98850 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98850/testReport)**
for PR 23038 at commit
Github user linhong-intel commented on the issue:
https://github.com/apache/spark/pull/23040
cc @xuanyuanking
---
-
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/23031
**[Test build #98858 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98858/testReport)**
for PR 23031 at commit
Github user ueshin commented on the issue:
https://github.com/apache/spark/pull/23031
@vanzin Thanks for letting me know it and that's really good to know.
I'll update this to use it.
---
-
To unsubscribe,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23040
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/23040
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/23040
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user linhong-intel opened a pull request:
https://github.com/apache/spark/pull/23040
[SPARK-26068][Core]ChunkedByteBufferInputStream should handle empty chunks
correctly
## What changes were proposed in this pull request?
Empty chunk in ChunkedByteBuffer will
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23034
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/23034
**[Test build #98857 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98857/testReport)**
for PR 23034 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22911
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/23034
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/22911
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/22989#discussion_r233713175
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/UnsafeAlignedOffset.java ---
@@ -39,7 +39,9 @@ public static int getSize(Object object, long
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/23033
---
-
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/23033
Merged to master.
---
-
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/23033
I am merging this for the same reason with #23021. Let me know if there's
any concern even after this got merged.
---
-
To
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/23033
@BryanCutler, looks we should add `pyspark.ml.tests` at
https://github.com/apache/spark/blob/master/python/run-tests.py#L252-L253 so
that we can run unittests first over doc tests (because
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22911
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98846/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22911
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/23038
**[Test build #98856 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98856/testReport)**
for PR 23038 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22911
**[Test build #98846 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98846/testReport)**
for PR 22911 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23033
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/23033
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98845/
Test PASSed.
---
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/23038
Retest this please
---
-
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/23033
**[Test build #98845 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98845/testReport)**
for PR 23033 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23034
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/23034
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98844/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23034
**[Test build #98844 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98844/testReport)**
for PR 23034 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23038
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98849/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23038
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/23038
**[Test build #98849 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98849/testReport)**
for PR 23038 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22989#discussion_r233706605
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/UnsafeAlignedOffset.java ---
@@ -52,7 +54,9 @@ public static void putSize(Object object,
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/22989#discussion_r233706517
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/UnsafeAlignedOffset.java ---
@@ -39,7 +39,9 @@ public static int getSize(Object object,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22911
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/22974
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/22911
Test FAILed.
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/22974
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/22974
**[Test build #98855 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98855/testReport)**
for PR 22974 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22946
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98852/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22946
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/22946
**[Test build #98852 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98852/testReport)**
for PR 22946 at commit
Github user suxingfate commented on the issue:
https://github.com/apache/spark/pull/22911
--keytab is a simpler way IMO to enable long running spark on k8s than DT
renew approach
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23034
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/23034
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98843/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23034
**[Test build #98843 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98843/testReport)**
for PR 23034 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22911
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/22911
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98842/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22911
**[Test build #98842 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98842/testReport)**
for PR 22911 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22946
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/22946
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98847/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22946
**[Test build #98847 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98847/testReport)**
for PR 22946 at commit
Github user AzureQ commented on the issue:
https://github.com/apache/spark/pull/23037
> I see this customization to be specific towards how you build your custom
Docker image. Unless it is relevant towards testing, we are trying to keep the
default Docker image as lightweight as
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23029
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/23029
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/23029
**[Test build #98854 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98854/testReport)**
for PR 23029 at commit
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23035#discussion_r233696401
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala
---
@@ -2554,4 +2554,34 @@ class DataFrameSuite extends QueryTest with
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/23035#discussion_r233695765
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/DataFrameSuite.scala
---
@@ -2554,4 +2554,34 @@ class DataFrameSuite extends QueryTest with
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23016
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/23016
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98853/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/23016
**[Test build #98853 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98853/testReport)**
for PR 23016 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23039
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98841/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23039
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/23039
**[Test build #98841 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98841/testReport)**
for PR 23039 at commit
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/23027#discussion_r233689556
--- Diff: sql/core/benchmarks/FilterPushdownBenchmark-results.txt ---
@@ -2,669 +2,809 @@
Pushdown for many distinct value case
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22911
Test FAILed.
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/22911
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user xuanyuanking commented on the issue:
https://github.com/apache/spark/pull/22989
Thanks @HyukjinKwon @viirya @felixcheung @srowen for your review and advise!
---
-
To unsubscribe, e-mail:
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/23027#discussion_r233687968
--- Diff: sql/core/benchmarks/FilterPushdownBenchmark-results.txt ---
@@ -2,669 +2,809 @@
Pushdown for many distinct value case
Github user wangyum commented on a diff in the pull request:
https://github.com/apache/spark/pull/23027#discussion_r233686986
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/FilterPushdownBenchmark.scala
---
@@ -131,6 +134,15 @@ object
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/23039
@MaxGekk . One PR should have one theme with a proper title. We frequently
search by commit title. Please don't split this PR into two sub PRs.
---
Github user xerial commented on the issue:
https://github.com/apache/spark/pull/22967
Thank you for all the efforts to make this happen!
Spark has been the last resort before deprecating Scala 2.11.
After Spark 3.0, as an OSS contributor, we can stop maintaining
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21688
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98839/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21688
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/21688
**[Test build #98839 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/98839/testReport)**
for PR 21688 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/23012
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/98848/
Test PASSed.
---
1 - 100 of 422 matches
Mail list logo
