Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21333#discussion_r196521913
--- Diff: core/src/test/scala/org/apache/spark/rdd/RDDSuite.scala ---
@@ -154,6 +154,13 @@ class RDDSuite extends SparkFunSuite with
SharedSparkContext {
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21564
thanks, merging to master!
---
-
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/21587
**[Test build #92094 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92094/testReport)**
for PR 21587 at commit
[`6fc7913`](https://github.com/apache/spark/commit/6f
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21587
Test FAILed.
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/4217/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21587
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/21587
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/21587
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/323/
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21531
---
-
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/21531
LGTM
Thanks! Merged to master/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional c
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/21577#discussion_r196514785
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/OutputCommitCoordinator.scala ---
@@ -109,20 +116,21 @@ private[spark] class OutputCommitCoordinato
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21577#discussion_r196514319
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/OutputCommitCoordinator.scala ---
@@ -109,20 +116,21 @@ private[spark] class OutputCommitCoordinator
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21577#discussion_r196513151
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/OutputCommitCoordinator.scala ---
@@ -109,20 +116,21 @@ private[spark] class OutputCommitCoordina
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21585
**[Test build #92093 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92093/testReport)**
for PR 21585 at commit
[`049844e`](https://github.com/apache/spark/commit/04
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21585
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/4216/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21585
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/21585
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/322/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21585
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user maryannxue commented on the issue:
https://github.com/apache/spark/pull/21585
Done with the changes. Thanks a lot, @cloud-fan !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21585
**[Test build #92092 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92092/testReport)**
for PR 21585 at commit
[`03c3c90`](https://github.com/apache/spark/commit/03
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21585
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/4215/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21585
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/21585
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/321/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21585
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21585#discussion_r196507229
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/sources/InsertSuite.scala ---
@@ -520,4 +544,29 @@ class InsertSuite extends DataSourceTest with
S
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21585
thanks, LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@s
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21585#discussion_r196506912
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/InsertIntoDataSourceCommand.scala
---
@@ -38,9 +38,8 @@ case class InsertInt
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21585
**[Test build #92091 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92091/testReport)**
for PR 21585 at commit
[`bb9fa03`](https://github.com/apache/spark/commit/bb
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21585
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/320/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21585
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/21585
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/21585
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/4214/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21591
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/21591
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/21591
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user misutoth opened a pull request:
https://github.com/apache/spark/pull/21591
[SQL][WIP] Added column name listing option to INSERT INTO
## What changes were proposed in this pull request?
Added possibility to specify column list to INSERT INTO. Source column list
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21577
I'm fine with separating them but we need a jira or need to update the v2
jira to handle all cases
---
-
To unsubscribe, e-mai
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/21577#discussion_r196494533
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/OutputCommitCoordinator.scala ---
@@ -109,20 +116,21 @@ private[spark] class OutputCommitCoordina
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21357
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/21357
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92088/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #92088 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92088/testReport)**
for PR 21357 at commit
[`55256b5`](https://github.com/apache/spark/commit/5
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/21590#discussion_r196490214
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCOptions.scala
---
@@ -65,13 +65,38 @@ class JDBCOptions(
/
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/21590#discussion_r196489749
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCOptions.scala
---
@@ -109,6 +134,20 @@ class JDBCOptions(
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20350
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92087/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20350
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/20350
**[Test build #92087 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92087/testReport)**
for PR 20350 at commit
[`069b861`](https://github.com/apache/spark/commit/0
Github user dilipbiswal commented on a diff in the pull request:
https://github.com/apache/spark/pull/21590#discussion_r196487627
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCOptions.scala
---
@@ -65,13 +65,38 @@ class JDBCOptions(
/
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21403
I updated the PR according to the previous discussion.
@hvanhovell @juliuszsompolski may you please take a look at it now? Thanks.
---
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21577#discussion_r196482353
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/OutputCommitCoordinator.scala ---
@@ -109,20 +116,21 @@ private[spark] class OutputCommitCoordinator
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21577#discussion_r196479588
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/OutputCommitCoordinator.scala ---
@@ -109,20 +116,21 @@ private[spark] class OutputCommitCoordi
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21577
This in general looks good, IMO we shall focus on fixing the output commit
coordinator issue in this PR, and discuss the data source issue in a separated
thread.
I'm OOO this week but will
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21588
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/21588
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92086/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21588
**[Test build #92086 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92086/testReport)**
for PR 21588 at commit
[`ef5ab0d`](https://github.com/apache/spark/commit/e
Github user bogdanrdc commented on the issue:
https://github.com/apache/spark/pull/21587
maybe also fix `SinglePartition.satisfies`. It is only checking for
ClusteredDistribution and defaults to true otherwise. Luckily,
`SinglePartition.numPartitions` is 1 so `EnsureRequirements` will
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/21556
cc @gatorsmile @rdblue
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21403
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/21403
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92084/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21403
**[Test build #92084 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92084/testReport)**
for PR 21403 at commit
[`df7d3ee`](https://github.com/apache/spark/commit/d
Github user wangyum commented on the issue:
https://github.com/apache/spark/pull/18424
@ptkool Are you still working on?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-m
Github user icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/21427#discussion_r196457433
--- Diff: python/pyspark/worker.py ---
@@ -110,9 +116,20 @@ def wrapped(key_series, value_series):
"Number of columns of the returned
Github user hthuynh2 commented on the issue:
https://github.com/apache/spark/pull/21527
I updated it. Thanks.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21527
I updated the jira description @hthuynh2 please update the description on
the PR to match
---
-
To unsubscribe, e-mail: revie
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21403
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92085/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21403
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/21403
**[Test build #92085 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92085/testReport)**
for PR 21403 at commit
[`c9a36e0`](https://github.com/apache/spark/commit/c
Github user icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/21427#discussion_r196442152
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/arrow/ArrowUtils.scala
---
@@ -120,4 +121,19 @@ object ArrowUtils {
StructFi
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21570
**[Test build #92090 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92090/testReport)**
for PR 21570 at commit
[`5be5a7a`](https://github.com/apache/spark/commit/5b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21570
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/4213/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21570
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/21570
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/21570
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/319/
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21427
Seems fine and I am okay with it.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21427#discussion_r196438132
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -1161,6 +1161,16 @@ object SQLConf {
.booleanConf
Github user icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/21427#discussion_r196437909
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/WindowInPandasExec.scala
---
@@ -97,7 +98,7 @@ case class WindowInPandasExec(
Github user icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/21427#discussion_r196437623
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/FlatMapGroupsInPandasExec.scala
---
@@ -77,7 +78,7 @@ case class FlatMapGroupsIn
Github user icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/21427#discussion_r196437348
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/ArrowPythonRunner.scala
---
@@ -58,18 +58,18 @@ class ArrowPythonRunner(
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21570#discussion_r196437321
--- Diff:
sql/core/src/test/java/test/org/apache/spark/sql/execution/sort/RecordBinaryComparatorSuite.java
---
@@ -0,0 +1,255 @@
+/*
+ * Licens
Github user icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/21427#discussion_r196436658
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/python/ArrowEvalPythonExec.scala
---
@@ -63,7 +64,7 @@ case class ArrowEvalPythonExec(u
Github user icexelloss commented on a diff in the pull request:
https://github.com/apache/spark/pull/21427#discussion_r196435526
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/arrow/ArrowUtils.scala
---
@@ -120,4 +121,19 @@ object ArrowUtils {
StructFi
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21577
So I think the commit/delete thing is also an issue for existing v1 and
hadoop committers as well. So this doesn't fully solve the problem. spark
uses a file format like
(HadoopMapReduceWriteCo
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/21558
Correct TaskId is unique. I agree with @squito can we rename to tid.
Hmm, good point @rdblue we need to make sure the output directories/files
and such would be unique. That problem ma
Github user jiangxb1987 commented on the issue:
https://github.com/apache/spark/pull/21567
Overall I don't think the current logic shall be modified. However, it
shall be useful to document some the configs mentioned in this PR.
---
--
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21109
**[Test build #92089 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92089/testReport)**
for PR 21109 at commit
[`9889ba1`](https://github.com/apache/spark/commit/98
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21567#discussion_r196433023
--- Diff: core/src/main/scala/org/apache/spark/ui/ConsoleProgressBar.scala
---
@@ -34,7 +34,7 @@ private[spark] class ConsoleProgressBar(sc: SparkContex
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21567#discussion_r196432597
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -613,7 +614,7 @@ private[spark] class Executor(
private[this] val
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21567#discussion_r196431492
--- Diff:
core/src/main/scala/org/apache/spark/deploy/worker/DriverRunner.scala ---
@@ -58,7 +59,7 @@ private[deploy] class DriverRunner(
/
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21567#discussion_r196431134
--- Diff:
core/src/main/scala/org/apache/spark/api/python/PythonRunner.scala ---
@@ -354,7 +355,8 @@ private[spark] abstract class BasePythonRunner[IN,
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21575#discussion_r196429048
--- Diff: core/src/main/scala/org/apache/spark/HeartbeatReceiver.scala ---
@@ -75,16 +76,18 @@ private[spark] class HeartbeatReceiver(sc:
SparkContext,
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21575#discussion_r196428732
--- Diff: core/src/main/scala/org/apache/spark/HeartbeatReceiver.scala ---
@@ -75,16 +76,18 @@ private[spark] class HeartbeatReceiver(sc:
SparkContext,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21589
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92081/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21589
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/21589
**[Test build #92081 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92081/testReport)**
for PR 21589 at commit
[`483743d`](https://github.com/apache/spark/commit/4
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21564
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/21564
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92082/
Test PASSed.
---
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/21333
cc @cloud-fan @JoshRosen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: re
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21564
**[Test build #92082 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92082/testReport)**
for PR 21564 at commit
[`dcd0ce9`](https://github.com/apache/spark/commit/d
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21590#discussion_r196407423
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCOptions.scala
---
@@ -65,13 +65,38 @@ class JDBCOptions(
// Req
Github user maropu commented on a diff in the pull request:
https://github.com/apache/spark/pull/21590#discussion_r196406329
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/JDBCOptions.scala
---
@@ -109,6 +134,20 @@ class JDBCOptions(
s"W
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21357
**[Test build #92088 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/92088/testReport)**
for PR 21357 at commit
[`55256b5`](https://github.com/apache/spark/commit/55
Github user HeartSaVioR commented on the issue:
https://github.com/apache/spark/pull/21357
retest this, please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
301 - 400 of 522 matches
Mail list logo
