Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20372
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/20372
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86591/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20208
**[Test build #86599 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86599/testReport)**
for PR 20208 at commit
[`29c281d`](https://github.com/apache/spark/commit/29
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20372
**[Test build #86591 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86591/testReport)**
for PR 20372 at commit
[`57722cf`](https://github.com/apache/spark/commit/5
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20208
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20387
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86598/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20387
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/20387
**[Test build #86598 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86598/testReport)**
for PR 20387 at commit
[`d3233e1`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20383
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86596/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20383
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/20383
**[Test build #86596 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86596/testReport)**
for PR 20383 at commit
[`cd9b73a`](https://github.com/apache/spark/commit/c
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20169
> I think you also notice that Wenchen and I are the top contributors and
reviewers in Spark SQL
No disagreement there.
> If any change made in Spark SQL, I think it would be nice
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20387
**[Test build #86598 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86598/testReport)**
for PR 20387 at commit
[`d3233e1`](https://github.com/apache/spark/commit/d3
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20387
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/20387
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/197/
Test
GitHub user rdblue opened a pull request:
https://github.com/apache/spark/pull/20387
SPARK-22386: DataSourceV2: Use immutable logical plans.
## What changes were proposed in this pull request?
DataSourceV2 should use immutable catalyst trees instead of wrapping a
mutable Da
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20169
@vanzin I never said I and Wenchen are acceptable peers. I have to correct
it. I think you also notice that Wenchen and I are the top contributors and
reviewers in Spark SQL. If any change made i
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20169
@gatorsmile, I see you merge changes with no review frequently, so I'm
surprised to see you say this. I trust your judgment when you do. You've made
the point repeatedly that you think you should be
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20385
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86597/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20385
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20169
And where exactly was a peer review missing here?
I didn't check this in without review. You're basically saying that only
you and Wenchen are acceptable "peers". I just disagree strongly wit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20385
**[Test build #86597 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86597/testReport)**
for PR 20385 at commit
[`c8fb436`](https://github.com/apache/spark/commit/c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20384
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86590/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20384
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/20384
**[Test build #86590 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86590/testReport)**
for PR 20384 at commit
[`3bbec21`](https://github.com/apache/spark/commit/3
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20169
@vanzin You might not understand my point. We need a peer review, even if
we are familiar with the code base. Let me give a better example for clarifying
my point.
Even if I am familiar
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/20367#discussion_r163640976
--- Diff:
mllib/src/main/scala/org/apache/spark/ml/feature/CountVectorizer.scala ---
@@ -113,7 +132,11 @@ private[feature] trait CountVectorizerParams extend
Github user MLnick commented on the issue:
https://github.com/apache/spark/pull/20363
Itâs just listing any breaking changes, if I missed them. Will do a pass
to
check and then remove WIP. If any folks know of breaking changes ping the
JIRA issue
On Wed, 24 Jan 2018 at 20
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20169
I think @vanzin is as qualified to modify this code as anyone. I defer to
your and his judgment about who really needs to weigh in, if anyone, before
merging. We don't, and can't, efficiently operate
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20161
Just pinging again to get some eyes.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20169
> I am not familiar with the code base of SHS
Please leave the strawmen out of the discussion. I *am* familiar with this
part of the code base, you just believe I'm not.
---
-
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20169
Yeah. That is a valid argument for the new contributors, but it is not true
for the active contributors and committers like you and me.
Let me give an example. I am not familiar with the
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20169
Yes I'm not following, @gatorsmile. You're saying there are no owners but
wanted to revert a change because you felt it was code that should only change
with your review. You're correct that there ar
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20385
**[Test build #86597 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86597/testReport)**
for PR 20385 at commit
[`c8fb436`](https://github.com/apache/spark/commit/c8
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20363
is this WIP?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20169
> That is why I suggested above to ping me and @cloud-fan when the
community does the change in Spark SQL.
The point I'm making (and I believe Hyukjin too) is that it should be the
opposite.
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18756
@lvdongr We still do not have a bandwidth to work on this issue. Could you
open a JIRA and ask the community for helping this? We need a design discussion
before starting the work.
---
---
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20385
Jenkins test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/19671
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user WeichenXu123 commented on the issue:
https://github.com/apache/spark/pull/19993
+1 merge this to 2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19671
Merging to master.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h..
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20169
Thank you for your previous contributions~ In the recent few releases,
there are major refactoring in the related codes. The codes do not belong to
anybody. Our goal is to build the best-quality
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20321
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user srowen commented on the issue:
https://github.com/apache/spark/pull/20321
Merged to master
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20359#discussion_r163631508
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/DriverRegistry.scala
---
@@ -32,6 +32,9 @@ import org.apache.spark.
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20377#discussion_r163631446
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -132,7 +131,8 @@ private[hive] class HiveClientImpl(
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/19671#discussion_r163631100
--- Diff:
core/src/test/scala/org/apache/spark/storage/BlockManagerSuite.scala ---
@@ -1322,33 +1322,55 @@ class BlockManagerSuite extends SparkFunSuite with
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20359#discussion_r163630976
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/DriverRegistry.scala
---
@@ -32,6 +32,9 @@ import org.apache.spark.uti
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20025
---
-
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/20377#discussion_r163630721
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -132,7 +131,8 @@ private[hive] class HiveClientImpl(
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20025
Thanks! Merged to master/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mai
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20169
> My only concern is we should notify people who wrote the code
Like, ahem, me? (Check this code history. I've worked on quite a good chunk
of it over time.)
I agree with what @Hyuk
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20376
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20377#discussion_r163628671
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -132,7 +131,8 @@ private[hive] class HiveClientImpl(
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20376
Merging to master / 2.3.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20377
@gatorsmile this fix is wrong. If you want to test for yourself, you need
to set `METASTOREWAREHOUSE` before instantiating the client in `VersionsSuite`.
---
---
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20369#discussion_r163628099
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/streaming/MicroBatchWriteSupport.java
---
@@ -1,60 +0,0 @@
-/*
- * Licensed to
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20377#discussion_r163627927
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -132,7 +131,8 @@ private[hive] class HiveClientImpl(
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20377#discussion_r163627455
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveClientImpl.scala
---
@@ -132,7 +131,8 @@ private[hive] class HiveClientImpl(
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20369#discussion_r163626800
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/streaming/DataStreamWriter.scala
---
@@ -281,11 +281,9 @@ final class DataStreamWriter[T] privat
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/18931
Before merging this PR, we need test cases.
- Add test cases to ensure our future changes will not break this.
- Add test cases to ensure the newly added conf
- Add boundary cases.
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20383
@zsxwing @jerryshao
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revie
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20383
**[Test build #86596 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86596/testReport)**
for PR 20383 at commit
[`cd9b73a`](https://github.com/apache/spark/commit/cd
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20383
Jenkins test this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20380
---
-
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/20203
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user felixcheung commented on the issue:
https://github.com/apache/spark/pull/20380
merged to master/2.3
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: revi
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20203
merged to master. thanks @attilapiros
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20386
**[Test build #86595 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86595/testReport)**
for PR 20386 at commit
[`11711a4`](https://github.com/apache/spark/commit/1
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20386
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86595/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20386
Merged build finished. Test FAILed.
---
-
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/18931#discussion_r163621072
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -1263,6 +1271,8 @@ class SQLConf extends Serializable with Loggi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20386
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/18931#discussion_r163620741
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -661,6 +661,14 @@ object SQLConf {
.intConf
.c
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20386
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/196/
Test
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20369#discussion_r163620226
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/WriteToDataSourceV2.scala
---
@@ -62,7 +62,7 @@ case class WriteToDataS
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20369#discussion_r163619964
--- Diff:
sql/core/src/main/java/org/apache/spark/sql/sources/v2/streaming/MicroBatchWriteSupport.java
---
@@ -1,60 +0,0 @@
-/*
- * Licensed to
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20386
**[Test build #86595 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86595/testReport)**
for PR 20386 at commit
[`11711a4`](https://github.com/apache/spark/commit/11
GitHub user gengliangwang opened a pull request:
https://github.com/apache/spark/pull/20386
[WIP][SPARK-23202][SQL] Break down DataSourceV2Writer.commit into two phase
## What changes were proposed in this pull request?
Currently, the api `DataSourceV2Writer#commit(WriterCom
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20382#discussion_r163614088
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/sources/TextSocketStreamSourceV2.scala
---
@@ -0,0 +1,247 @@
+/*
+ *
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20382#discussion_r163613655
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/sources/TextSocketStreamSourceV2.scala
---
@@ -0,0 +1,247 @@
+/*
+ *
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20382#discussion_r163616762
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/sources/TextSocketStreamSourceV2.scala
---
@@ -0,0 +1,247 @@
+/*
+ *
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/20382#discussion_r163612181
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/sources/TextSocketStreamSourceV2.scala
---
@@ -0,0 +1,247 @@
+/*
+ *
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19671
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/19671
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86587/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20376
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/86588/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19671
**[Test build #86587 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86587/testReport)**
for PR 19671 at commit
[`5319ae3`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20376
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/20376
**[Test build #86588 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86588/testReport)**
for PR 20376 at commit
[`dfd828b`](https://github.com/apache/spark/commit/d
Github user liufengdb commented on the issue:
https://github.com/apache/spark/pull/20025
lgtm
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apa
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20359#discussion_r163614580
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/DriverRegistry.scala
---
@@ -32,6 +32,9 @@ import org.apache.spark.
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20384
**[Test build #86594 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86594/testReport)**
for PR 20384 at commit
[`aa33da3`](https://github.com/apache/spark/commit/aa
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/20359#discussion_r163613813
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/jdbc/DriverRegistry.scala
---
@@ -32,6 +32,9 @@ import org.apache.spark.
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20384
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/195/
Test
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20384
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20381
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20368#discussion_r163610413
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/joins/BroadcastJoinSuite.scala
---
@@ -126,6 +126,22 @@ class BroadcastJoinSuite extend
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20375
**[Test build #86593 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86593/testReport)**
for PR 20375 at commit
[`804e506`](https://github.com/apache/spark/commit/80
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20381
**[Test build #86592 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/86592/testReport)**
for PR 20381 at commit
[`6869155`](https://github.com/apache/spark/commit/68
Github user gatorsmile commented on the issue:
https://github.com/apache/spark/pull/20381
ok to test
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@sp
301 - 400 of 592 matches
Mail list logo