Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22484
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3601/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22484
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22484
**[Test build #96811 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96811/testReport)**
for PR 22484 at commit
[`6c46ad5`](https://github.com/apache/spark/commit/6c
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22484#discussion_r221503808
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/SqlBasedBenchmark.scala
---
@@ -0,0 +1,60 @@
+/*
+ * Licensed to t
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22484#discussion_r221501996
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/SqlBasedBenchmark.scala
---
@@ -0,0 +1,60 @@
+/*
+ * Licensed to t
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22484#discussion_r221501890
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/SqlBasedBenchmark.scala
---
@@ -0,0 +1,60 @@
+/*
+ * Licensed to t
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22484#discussion_r221501761
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/SqlBasedBenchmark.scala
---
@@ -0,0 +1,60 @@
+/*
+ * Licensed to t
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22461
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22461
Thanks! Merged to master/2.4
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mai
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22593#discussion_r221497310
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/DataStreamWriter.scala
---
@@ -46,14 +46,16 @@ final class DataStreamWriter[T] private
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22573
That's great!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22582
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22573
Updating `Filter` APIs sounds reasonable to me. This should be part of our
data source API v2. cc @cloud-fan @rxin @rdblue
---
-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22524
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22524
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96807/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22524
**[Test build #96807 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96807/testReport)**
for PR 22524 at commit
[`ed2c269`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22597
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3600/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22597
**[Test build #96810 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96810/testReport)**
for PR 22597 at commit
[`f6c3dca`](https://github.com/apache/spark/commit/f6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22597
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/22597
[SPARK-25579][SQL] Use quoted attribute names if needed in pushed ORC
predicates
## What changes were proposed in this pull request?
This PR aims to fix an ORC performance regression
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22596
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22596
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96808/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22596
**[Test build #96808 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96808/testReport)**
for PR 22596 at commit
[`c4b6920`](https://github.com/apache/spark/commit/c
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22527
**[Test build #96809 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96809/testReport)**
for PR 22527 at commit
[`d8083cf`](https://github.com/apache/spark/commit/d8
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/22527
Jenkins, ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22596
Can you link the JIRA https://issues.apache.org/jira/browse/SPARK-25576 ?
Please see https://spark.apache.org/contributing.html
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22596
**[Test build #96808 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96808/testReport)**
for PR 22596 at commit
[`c4b6920`](https://github.com/apache/spark/commit/c4
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22596
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22539
Looks https://github.com/apache/spark/pull/21363 getting inactive. Can you
take this over instead? You can pick up the commits there and open another PR.
---
--
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22524
**[Test build #96807 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96807/testReport)**
for PR 22524 at commit
[`ed2c269`](https://github.com/apache/spark/commit/ed
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22524
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3599/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22524
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22573
Can we update public `Filter` API in Spark 3.0.0? @cloud-fan and
@gatorsmile .
---
-
To unsubscribe, e-mail: reviews-unsub
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22573#discussion_r221477177
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -437,53 +436,65 @@ object DataSourceStra
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/22573#discussion_r221476951
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/DataSourceStrategy.scala
---
@@ -437,53 +436,65 @@ object DataSourceStra
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22138
Kindly ask for reviewing. Please never mind when you're busy with fixing
bugs on Spark 2.4 RC.
@gaborgsomogyi I guess I left two things for committer decision: 1. define
soft boundary
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22565
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22565
Merged to master/branch-2.4.
Thank you, @ScrapCodes and @liyinan926 .
---
-
To unsubscribe, e-mail: reviews-unsubs
Github user szhem commented on the issue:
https://github.com/apache/spark/pull/19410
Hello @mallman, @sujithjay, @felixcheung, @jkbradley, @mengxr, it's already
about a year passed since this pull request has been opened.
I'm just wondering whether there is any chance to get any fe
Github user szhem commented on the issue:
https://github.com/apache/spark/pull/19373
Hello @sujithjay, @felixcheung, @jkbradley, @mengxr, it's already more than
a year passed since this pull request has been opened.
I'm just wondering whether there is any chance for this PR to be
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21221
@mccheah When you merged the code, could you also leave the comments about
which branches you did the merge?
---
-
To unsubs
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21221
For the other reviewers, this was merged to master (not 2.4)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.or
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22596
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/22596
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/22596
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user samdvr opened a pull request:
https://github.com/apache/spark/pull/22596
Fix lint failure in 2.2
## What changes were proposed in this pull request?
Line length fixes and
## How was this patch tested?
Manually verified, but will ensure jenkins
Github user softmanu commented on the issue:
https://github.com/apache/spark/pull/22539
@dongjoon-hyun @HyukjinKwon
Hi,
i was not well whole last week, now I am back, so yes, thanks for reviewing
and all the comments. whether my PR is a duplicate or not we can see out later,
Github user shahidki31 commented on the issue:
https://github.com/apache/spark/pull/22592
Thank you @srowen .
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22577
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user suryag10 commented on the issue:
https://github.com/apache/spark/pull/21669
> like it, but we could also first support cluster mode and add client mode
after.
Thats the reason i said "Point to note":)-
---
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22577
Merged to master/2.4/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
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 comma
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/96806/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22594
**[Test build #96806 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96806/testReport)**
for PR 22594 at commit
[`e589e1e`](https://github.com/apache/spark/commit/e
Github user LantaoJin commented on the issue:
https://github.com/apache/spark/pull/22595
If this PR could be merged, #22578 could be added as an additional column
as well.
---
-
To unsubscribe, e-mail: reviews-unsub
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22595
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/22595
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user LantaoJin commented on the issue:
https://github.com/apache/spark/pull/22595
Gently ping @jerryshao @cloud-fan . Do you have a chance to review?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.ap
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22595
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user LantaoJin opened a pull request:
https://github.com/apache/spark/pull/22595
[SPARK-25577][Web UI] Add an on-off switch to display the executor
additional columns
## What changes were proposed in this pull request?
[SPARK-17019](https://issues.apache.org/jira/bro
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22594
**[Test build #96806 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96806/testReport)**
for PR 22594 at commit
[`e589e1e`](https://github.com/apache/spark/commit/e5
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 comma
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/testing-k8s-prb-make-spark-distribution-unified/3598/
GitHub user 10110346 opened a pull request:
https://github.com/apache/spark/pull/22594
[MINOR][SQL] When batch reading, the number of bytes can not be updated as
expected.
## What changes were proposed in this pull request?
When batch reading, the number of bytes can not be
64 matches
Mail list logo