Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22009
**[Test build #95033 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95033/testReport)**
for PR 22009 at commit
[`f938614`](https://github.com/apache/spark/commit/f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22009
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95033/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22009
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/21977
@holdenk pyarrow uses a C++ based memory pool, so I'm not sure how that
exactly works with rlimit but I ran some tests and looks like an error is
thrown when the limit is set.
**with se
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21756
**[Test build #95049 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95049/testReport)**
for PR 21756 at commit
[`5234c98`](https://github.com/apache/spark/commit/52
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/22160
Yeah that's right, we have had two parallel trees for different Scala
versions just for reasons like this. There is less forked code now than with
2.10 vs 2.11. If you think we can merge all the code
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21976
Please hold off on merging this am investigating a weirdness I want to make
sure isn't caused by this.
---
-
To unsubscribe, e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22174
**[Test build #4284 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4284/testReport)**
for PR 22174 at commit
[`28ea230`](https://github.com/apache/spark/commit/2
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22112
**[Test build #95023 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95023/testReport)**
for PR 22112 at commit
[`4f8e24d`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22112
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/22112
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95023/
Test FAILed.
---
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22174#discussion_r211727767
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -1954,14 +1954,7 @@ class SQLConf extends Serializable with Logg
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22154#discussion_r211729173
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CodeGeneratorWithInterpretedFallback.scala
---
@@ -63,7 +49,10 @@ abstra
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22154#discussion_r211729573
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CodeGeneratorWithInterpretedFallback.scala
---
@@ -63,7 +49,10 @@ abstra
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22154#discussion_r211730445
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/Projection.scala
---
@@ -180,7 +180,10 @@ object UnsafeProjection
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22154#discussion_r211730772
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CodeGeneratorWithInterpretedFallback.scala
---
@@ -17,24 +17,10 @@
Github user rednaxelafx commented on a diff in the pull request:
https://github.com/apache/spark/pull/22154#discussion_r211731392
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CodeGeneratorWithInterpretedFallback.scala
---
@@ -63,7 +49,10 @@ abstr
Github user rednaxelafx commented on a diff in the pull request:
https://github.com/apache/spark/pull/22154#discussion_r211731624
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CodeGeneratorWithInterpretedFallback.scala
---
@@ -63,7 +49,10 @@ abstr
Github user rednaxelafx commented on a diff in the pull request:
https://github.com/apache/spark/pull/22154#discussion_r211731921
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/CodeGeneratorWithInterpretedFallback.scala
---
@@ -17,24 +17,10 @@
Github user icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/22104#discussion_r211733007
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/EvalPythonExec.scala
---
@@ -117,15 +117,18 @@ abstract class EvalPythonExec(udf
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r211733592
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeValidator.scala
---
@@ -0,0 +1,185 @@
+/*
+ * Licensed to t
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20611
**[Test build #95030 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95030/testReport)**
for PR 20611 at commit
[`b410f99`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21977
**[Test build #95050 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95050/testReport)**
for PR 21977 at commit
[`505f2eb`](https://github.com/apache/spark/commit/50
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20611
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/20611
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95030/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21977
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/21977
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/2396/
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/21977
@BryanCutler, thanks for taking a look at this.
Despite the problems you hit when the limit was set too low, I think we do
want to use that limit. It was the most reliable one from our testin
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/22009#discussion_r211739221
--- Diff:
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaContinuousTest.scala
---
@@ -47,7 +47,9 @@ trait KafkaContinuousTes
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/22140
Does it make any sense to have less values than fields? Maybe we should
check that they are equal, wdyt @HyukjinKwon ?
---
-
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21977#discussion_r211752765
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala
---
@@ -91,6 +91,13 @@ private[spark] class Client(
private
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21977#discussion_r211750476
--- Diff:
core/src/main/scala/org/apache/spark/api/python/PythonRunner.scala ---
@@ -60,14 +61,20 @@ private[spark] object PythonEvalType {
*/
pri
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21977#discussion_r211748871
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/BaseYarnClusterSuite.scala
---
@@ -161,6 +162,11 @@ abstract class BaseYarnClus
Github user rajeshcode commented on the issue:
https://github.com/apache/spark/pull/17342
Sorry about above stack. The actual error is as below. Thats why relates
to SPARK-21697
Removing commons-logging jar fixes it. copying back common-logging jar
never complains again that
Github user steveloughran commented on the issue:
https://github.com/apache/spark/pull/17342
Well, no obvious answer there I'm afraid, except "don't put HDFS JARs on
the classpath"; if you serve them up via HTTP all should work
---
---
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/21977#discussion_r211763465
--- Diff:
core/src/main/scala/org/apache/spark/api/python/PythonRunner.scala ---
@@ -60,14 +61,20 @@ private[spark] object PythonEvalType {
*/
pri
Github user rdblue commented on a diff in the pull request:
https://github.com/apache/spark/pull/21977#discussion_r211763717
--- Diff:
resource-managers/yarn/src/test/scala/org/apache/spark/deploy/yarn/BaseYarnClusterSuite.scala
---
@@ -161,6 +162,11 @@ abstract class BaseYarnClus
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/22170
@HyukjinKwon Thanks for the reviewing and merging!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For add
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/22009
@cloud-fan, I think that the scan config builder needs to accept the
options and that SaveMode needs to be removed before we should merge this PR.
I'm fine with following up with the WriteConfig addi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21320
**[Test build #95041 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95041/testReport)**
for PR 21320 at commit
[`2711746`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21320
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/21320
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95041/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22135
**[Test build #95038 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95038/testReport)**
for PR 22135 at commit
[`643d432`](https://github.com/apache/spark/commit/6
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22135
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95038/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22135
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/22085
**[Test build #95037 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95037/testReport)**
for PR 22085 at commit
[`2a8f3cb`](https://github.com/apache/spark/commit/2
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22085
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/22085
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95037/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22085
**[Test build #95040 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95040/testReport)**
for PR 22085 at commit
[`1cacd40`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22085
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/22085
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95040/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21976
**[Test build #95034 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95034/testReport)**
for PR 21976 at commit
[`e384245`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21976
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/21976
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95034/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20637
**[Test build #95044 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95044/testReport)**
for PR 20637 at commit
[`d379718`](https://github.com/apache/spark/commit/d
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/21756
I don't understand the purpose of this change. Can you explain exactly what
part of `SparkHadoopUtil` do you need to customize, and why that change can't
just be made to the shared code?
The
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20637
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95044/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20637
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/22133
+1
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apach
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/22121#discussion_r211773604
--- Diff: docs/avro-data-source-guide.md ---
@@ -0,0 +1,260 @@
+---
+layout: global
+title: Apache Avro Data Source Guide
+---
+
+*
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/16478
retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16478
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/2397/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16478
**[Test build #95051 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95051/testReport)**
for PR 16478 at commit
[`ae00de1`](https://github.com/apache/spark/commit/ae
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16478
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user jkbradley commented on the issue:
https://github.com/apache/spark/pull/22139
LGTM
I'll merge this with master
Thanks @MrBago !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Fo
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16478
**[Test build #95051 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95051/testReport)**
for PR 16478 at commit
[`ae00de1`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16478
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/16478
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95051/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22063
**[Test build #95031 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95031/testReport)**
for PR 22063 at commit
[`a5cc5ec`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22063
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95031/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22063
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22139
---
-
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/21733
---
-
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/21749
**[Test build #95052 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95052/testReport)**
for PR 21749 at commit
[`8715fbe`](https://github.com/apache/spark/commit/87
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/21469
LGTM.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache
Github user tdas commented on the issue:
https://github.com/apache/spark/pull/21469
Merged to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22133
---
-
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/21469
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21749
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/21749
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/2398/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21912
**[Test build #95046 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95046/testReport)**
for PR 21912 at commit
[`a79d506`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21912
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95046/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21912
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21733
Thanks all for reviewing and thanks @tdas for merging this in!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21469
Thanks all for reviewing and thanks @tdas for merging this in!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/22148#discussion_r211783541
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/FileBasedDataSourceSuite.scala ---
@@ -430,6 +430,48 @@ class FileBasedDataSourceSuite extends Que
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22154
**[Test build #95047 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95047/testReport)**
for PR 22154 at commit
[`5481255`](https://github.com/apache/spark/commit/5
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22148
@seancxmao Please submit a follow-up PR to document the behavior changes in
the migration guide of Spark SQL?
---
-
To unsubs
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22154
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/22154
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/95047/
Test PASSed.
---
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/22079
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22079
**[Test build #95053 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95053/testReport)**
for PR 22079 at commit
[`2edad85`](https://github.com/apache/spark/commit/2e
Github user mengxr commented on the issue:
https://github.com/apache/spark/pull/22085
LGTM. I'm merging this into master. We might need a minor refactor for
readability. But it shouldn't block developers testing this new feature. Thanks!
---
-
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22085
---
-
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/22079
**[Test build #95054 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95054/testReport)**
for PR 22079 at commit
[`2edad85`](https://github.com/apache/spark/commit/2e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21546
**[Test build #95055 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/95055/testReport)**
for PR 21546 at commit
[`92b8e26`](https://github.com/apache/spark/commit/92
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/22042#discussion_r211786163
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaDataConsumer.scala
---
@@ -31,22 +31,21 @@ import
org.apache.spark.sql
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/22042#discussion_r211786183
--- Diff:
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaDataConsumer.scala
---
@@ -31,22 +31,21 @@ import
org.apache.spark.sql
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21546
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/2399/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21546
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
401 - 500 of 793 matches
Mail list logo
