Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21311#discussion_r190106818
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/joins/HashedRelationSuite.scala
---
@@ -254,6 +254,30 @@ class HashedRelationSuite exten
Github user jerryshao commented on the issue:
https://github.com/apache/spark/pull/21396
Can you please describe your scenario @LantaoJin ?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For addit
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/21390#discussion_r190105740
--- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala
---
@@ -97,6 +97,10 @@ private[deploy] class Worker(
private val APP_DATA
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21311
**[Test build #91009 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91009/testReport)**
for PR 21311 at commit
[`b8b6324`](https://github.com/apache/spark/commit/b8
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21311
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h..
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/21402
to confirm, `StreamRequest` will not block the server netty handler thread,
right?
---
-
To unsubscribe, e-mail: reviews-unsub
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21372
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90999/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21372
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/21404
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/21404
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91003/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21372
**[Test build #90999 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90999/testReport)**
for PR 21372 at commit
[`954d1d9`](https://github.com/apache/spark/commit/9
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21404
**[Test build #91003 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91003/testReport)**
for PR 21404 at commit
[`079771d`](https://github.com/apache/spark/commit/0
Github user JoshRosen commented on a diff in the pull request:
https://github.com/apache/spark/pull/21390#discussion_r190104118
--- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala
---
@@ -732,6 +736,9 @@ private[deploy] class Worker(
trimFinishe
Github user galv commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r190104076
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/Client.scala
---
@@ -249,6 +259,10 @@ private[spark] class Client(
val c
Github user galv commented on a diff in the pull request:
https://github.com/apache/spark/pull/20761#discussion_r190103364
--- Diff:
resource-managers/yarn/src/main/scala/org/apache/spark/deploy/yarn/ResourceTypeHelper.scala
---
@@ -0,0 +1,180 @@
+/*
+ * Licensed to the Ap
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21404
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/21404
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/3485/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21404
**[Test build #91008 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91008/testReport)**
for PR 21404 at commit
[`079771d`](https://github.com/apache/spark/commit/07
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21404
Retest this please.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: rev
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21404
The failures some from `ClassNotFoundException:
org.apache.logging.log4j.util.Strings`.
---
-
To unsubscribe, e-mail: revi
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/21404#discussion_r190101719
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
@@ -1148,3 +1149,128 @@ private[client] class Shim_v2_1 extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21082
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90996/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21082
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user LantaoJin commented on the issue:
https://github.com/apache/spark/pull/21396
[#20784](https://github.com/apache/spark/pull/20784) and
[#21343](https://github.com/apache/spark/pull/21343) did the same thing, but
#21343 is much readable. They are all to fix the problem using
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21082
**[Test build #90996 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90996/testReport)**
for PR 21082 at commit
[`ed02823`](https://github.com/apache/spark/commit/e
Github user LantaoJin commented on a diff in the pull request:
https://github.com/apache/spark/pull/21396#discussion_r190100259
--- Diff:
core/src/main/scala/org/apache/spark/deploy/security/HiveDelegationTokenProvider.scala
---
@@ -85,7 +85,10 @@ private[spark] class HiveDelegati
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21069
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/3484/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21069
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
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/21403
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90998/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21403
**[Test build #90998 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90998/testReport)**
for PR 21403 at commit
[`5b6226f`](https://github.com/apache/spark/commit/5
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/21387
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21390#discussion_r190098764
--- Diff:
common/network-shuffle/src/test/java/org/apache/spark/network/shuffle/NonShuffleFilesCleanupSuite.java
---
@@ -0,0 +1,222 @@
+/*
+ *
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21387
Merged to master and branch-2.3.
---
-
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/21069
**[Test build #91007 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91007/testReport)**
for PR 21069 at commit
[`9281ae2`](https://github.com/apache/spark/commit/92
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21390#discussion_r190098624
--- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala
---
@@ -732,6 +736,9 @@ private[deploy] class Worker(
trimFinis
Github user huaxingao commented on the issue:
https://github.com/apache/spark/pull/21069
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21390
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/21390
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/3483/
Tes
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21390#discussion_r190098033
--- Diff: core/src/main/scala/org/apache/spark/deploy/worker/Worker.scala
---
@@ -97,6 +97,10 @@ private[deploy] class Worker(
private val APP_DA
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21404
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91001/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21404
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/21404
**[Test build #91001 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91001/testReport)**
for PR 21404 at commit
[`1523b65`](https://github.com/apache/spark/commit/1
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/20929
@mengxr Could you check? Thanks!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mai
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21390
**[Test build #91006 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91006/testReport)**
for PR 21390 at commit
[`0df8e4e`](https://github.com/apache/spark/commit/0d
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21401
**[Test build #91005 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91005/testReport)**
for PR 21401 at commit
[`8e8f623`](https://github.com/apache/spark/commit/8e
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/21401
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h..
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/21404#discussion_r190097176
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/HiveShim.scala ---
@@ -1148,3 +1149,128 @@ private[client] class Shim_v2_1 extends Sh
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21394
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/21394
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90997/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21394
**[Test build #90997 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90997/testReport)**
for PR 21394 at commit
[`70839e8`](https://github.com/apache/spark/commit/7
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21045
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/21045
**[Test build #91004 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91004/testReport)**
for PR 21045 at commit
[`e3bf83e`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21045
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91004/
Test FAILed.
---
Github user DylanGuedes commented on a diff in the pull request:
https://github.com/apache/spark/pull/21045#discussion_r190095025
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -127,6 +127,148 @@ case class MapKeys
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21045
**[Test build #91004 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91004/testReport)**
for PR 21045 at commit
[`e3bf83e`](https://github.com/apache/spark/commit/e3
Github user DylanGuedes commented on a diff in the pull request:
https://github.com/apache/spark/pull/21045#discussion_r190094810
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -288,6 +289,88 @@ class Collect
Github user DylanGuedes commented on a diff in the pull request:
https://github.com/apache/spark/pull/21045#discussion_r190094430
--- Diff:
sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/expressions/CollectionExpressionsSuite.scala
---
@@ -288,6 +289,88 @@ class Collect
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/21397
Only when udf raises error. In normal case, there is no race because the
writer thread always closes the root and allocator before task completion
listener runs.
---
--
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/20208
Sorry for the delay. I updated the PR according to the comments,
@gatorsmile .
Could you review this once more?
---
-
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21404
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/21404
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/3482/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21404
**[Test build #91003 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91003/testReport)**
for PR 21404 at commit
[`079771d`](https://github.com/apache/spark/commit/07
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/21397
One more question, do you only observe this when the python udf raises an
error or have you seen it in normal runtime operation?
---
---
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/21404#discussion_r190088876
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/package.scala ---
@@ -79,7 +79,12 @@ package object client {
exclusions
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21385
**[Test build #91002 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91002/testReport)**
for PR 21385 at commit
[`ca3f5cd`](https://github.com/apache/spark/commit/ca
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/21385#discussion_r190088531
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/shuffle/UnsafeRowReceiver.scala
---
@@ -56,20 +62,46 @@ private[s
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/21385
addressed comments
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/21385#discussion_r190088467
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/continuous/shuffle/UnsafeRowReceiver.scala
---
@@ -56,20 +62,46 @@ private[s
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21404#discussion_r190086082
--- Diff:
sql/hive/src/main/scala/org/apache/spark/sql/hive/client/package.scala ---
@@ -79,7 +79,12 @@ package object client {
exclusions = S
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21404
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/3481/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21404
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/20894
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90993/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20894
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/20894
**[Test build #90993 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90993/testReport)**
for PR 20894 at commit
[`7dce1e7`](https://github.com/apache/spark/commit/7
Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/21385#discussion_r190082630
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/continuous/shuffle/ContinuousShuffleReadSuite.scala
---
@@ -161,13 +189,15 @@ class Continuou
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21404
**[Test build #91001 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91001/testReport)**
for PR 21404 at commit
[`1523b65`](https://github.com/apache/spark/commit/15
GitHub user dongjoon-hyun opened a pull request:
https://github.com/apache/spark/pull/21404
[SPARK-24360][SQL] Support Hive 3.0 metastore
## What changes were proposed in this pull request?
Hive 3.0 is released. This PR aims to support Hive 3.0 metastore.
## How was
Github user icexelloss commented on the issue:
https://github.com/apache/spark/pull/21397
Yeah this is a bit tricky because of the race. The test does fail on my
machine without the fix.
I have been changing the test data size until I can reproduce it, so it's
not great. If y
Github user jose-torres commented on a diff in the pull request:
https://github.com/apache/spark/pull/21385#discussion_r190080895
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/streaming/continuous/shuffle/ContinuousShuffleReadSuite.scala
---
@@ -161,13 +189,15 @@ class Co
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21398
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90994/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21398
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/21398
**[Test build #90994 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90994/testReport)**
for PR 21398 at commit
[`5f64a95`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21399
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/3480/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21399
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/21397
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/21397
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90991/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21397
**[Test build #90991 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90991/testReport)**
for PR 21397 at commit
[`69c9104`](https://github.com/apache/spark/commit/6
Github user liyinan926 commented on the issue:
https://github.com/apache/spark/pull/21260
@andrusha Is this ready for review?
---
-
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/21399
**[Test build #91000 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91000/testReport)**
for PR 21399 at commit
[`7bb0eb3`](https://github.com/apache/spark/commit/7b
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/21399
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: review
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/21397
@icexelloss the test you added passes for me without the fix, but I did see
a message of a suppressed exception in the finally block
"java.lang.IllegalStateException: ArrowBuf[3] refCnt has gone
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/21295#discussion_r190076831
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/parquet/ParquetQuerySuite.scala
---
@@ -879,6 +879,18 @@ class ParquetQuery
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21390#discussion_r190074813
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/util/JavaUtils.java
---
@@ -157,10 +172,10 @@ private static void
deleteRecursi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21372
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/3479/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21372
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/21372
**[Test build #90999 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/90999/testReport)**
for PR 21372 at commit
[`954d1d9`](https://github.com/apache/spark/commit/95
Github user dongjoon-hyun commented on a diff in the pull request:
https://github.com/apache/spark/pull/21372#discussion_r190073105
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/orc/OrcSourceSuite.scala
---
@@ -169,6 +170,14 @@ abstract class OrcSuit
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/testing-k8s-prb-make-spark-distribution/3478/
Tes
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
101 - 200 of 822 matches
Mail list logo