Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21547
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/4012/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21560
**[Test build #91817 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91817/testReport)**
for PR 21560 at commit
[`252f5c9`](https://github.com/apache/spark/commit/25
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/21288#discussion_r195305634
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/benchmark/FilterPushdownBenchmark.scala
---
@@ -131,211 +132,214 @@ object FilterPus
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21537#discussion_r195306721
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -579,6 +579,22 @@ class CodegenContext
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21537#discussion_r195306844
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -579,6 +579,22 @@ class CodegenContext
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/21537#discussion_r195307036
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/codegen/CodeGenerator.scala
---
@@ -579,6 +579,22 @@ class CodegenContext {
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21503
cc @rxin if you are interested.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21441
**[Test build #91809 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91809/testReport)**
for PR 21441 at commit
[`f16c7f7`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21441
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91809/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21441
Merged build finished. Test FAILed.
---
-
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/21389
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/21441
**[Test build #91810 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91810/testReport)**
for PR 21441 at commit
[`f16c7f7`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21441
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91810/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21441
Merged build finished. Test FAILed.
---
-
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/21535
**[Test build #91807 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91807/testReport)**
for PR 21535 at commit
[`250074b`](https://github.com/apache/spark/commit/2
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21288
@maropu Could you fix the style?
BTW, based on the latest result, Parquet is generally faster than ORC. cc
@dongjoon-hyun @rdblue
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21535
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/21535
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91807/
Test PASSed.
---
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21092
@lucashu1 please send your question to stackoverflow or
u...@spark.apache.org!
---
-
To unsubscribe, e-mail: reviews-unsubsc
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21533#discussion_r195310633
--- Diff: core/src/main/scala/org/apache/spark/SparkContext.scala ---
@@ -1517,9 +1517,12 @@ class SparkContext(config: SparkConf) extends
Logging {
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/21531
Jenkins, retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mai
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21533
LGTM except for the comment from @HyukjinKwon
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additio
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21536
@HyukjinKwon can we merge this?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21531
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/4013/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21531
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/21531
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/21531
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/124/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21531
**[Test build #91819 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91819/testReport)**
for PR 21531 at commit
[`c9db68d`](https://github.com/apache/spark/commit/c9
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21536
Sure.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20641
ok, could you rebase this PR
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20641
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20641
@mgummelt too
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h..
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20641
**[Test build #91820 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91820/testReport)**
for PR 20641 at commit
[`22c6739`](https://github.com/apache/spark/commit/22
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21548
CC @vanzin @tgravescs would you please help to review, thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.o
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21288
ok
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.o
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21536
Merged to master and branch-2.3.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21536
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21379#discussion_r195313083
--- Diff: sql/core/src/test/scala/org/apache/spark/sql/jdbc/JDBCSuite.scala
---
@@ -1206,4 +1211,47 @@ class JDBCSuite extends SparkFunSuite
}.g
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21288
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/21288
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/125/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21288
**[Test build #91821 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91821/testReport)**
for PR 21288 at commit
[`d3dd504`](https://github.com/apache/spark/commit/d3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21288
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/4014/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21288
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21379#discussion_r195313934
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRelation.scala
---
@@ -51,7 +53,11 @@ private[sql] object JDBCRela
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21379#discussion_r195313861
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRelation.scala
---
@@ -99,10 +115,21 @@ private[sql] object JDBCRe
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21379#discussion_r195314034
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRelation.scala
---
@@ -78,7 +84,17 @@ private[sql] object JDBCRela
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21379
@maropu The fix looks good to me. Thanks for changing the solution. Could
you update the PR title and description?
---
-
To u
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21379#discussion_r195315740
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCRelation.scala
---
@@ -51,7 +53,11 @@ private[sql] object JDBCRelation
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21379
oh, I forgot to update the title..yea, I'll do soon.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additi
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21536
Thanks! @HyukjinKwon
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21370#discussion_r195316251
--- Diff: docs/configuration.md ---
@@ -456,6 +456,33 @@ Apart from these, the following properties are also
available, and may be useful
from J
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/21553
Could you address the comments in the original PR?
- https://github.com/apache/spark/pull/21370#discussion_r194276735
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21288
@gatorsmile and @maropu . I really appreciate this effort. Thanks.
Since this is a cloud benchmark, I have one thing to recommend. Can we use
`r3.xlarge` for all benchmarks **consisten
601 - 653 of 653 matches
Mail list logo