Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20653
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/19788
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/20653
**[Test build #87612 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87612/testReport)**
for PR 20653 at commit
[`b5c8d56`](https://github.com/apache/spark/commit/b
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19788
**[Test build #87610 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87610/testReport)**
for PR 19788 at commit
[`583666b`](https://github.com/apache/spark/commit/5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20653
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87612/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19788
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87610/
Test FAILed.
---
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20649#discussion_r169882987
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/StructTypeSuite.scala ---
@@ -0,0 +1,43 @@
+/*
+ * Licensed to the Apache Software Founda
Github user yucai commented on the issue:
https://github.com/apache/spark/pull/19788
@cloud-fan if encryption is enabled
`blockManager.serializerManager().encryptionEnabled() == true`, shall we
disable this feature also?
---
-
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19788#discussion_r169898393
--- Diff: project/MimaExcludes.scala ---
@@ -1129,6 +1129,12 @@ object MimaExcludes {
ProblemFilters.exclude[InheritedNewAbstractMethodProblem]
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19788#discussion_r169899114
--- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
@@ -295,8 +307,8 @@ private[spark] abstract class MapOutputTracker(conf:
SparkCo
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20651
@zsxwing SKIPPED stages were previously shown as PENDING. Despite I agree
that your fix is needed, I think this is needed too. Do you agree @vanzin?
---
-
Github user yucai commented on a diff in the pull request:
https://github.com/apache/spark/pull/19788#discussion_r169901683
--- Diff: project/MimaExcludes.scala ---
@@ -1129,6 +1129,12 @@ object MimaExcludes {
ProblemFilters.exclude[InheritedNewAbstractMethodProblem]("or
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19788#discussion_r169902931
--- Diff: core/src/main/scala/org/apache/spark/MapOutputTracker.scala ---
@@ -295,8 +307,8 @@ private[spark] abstract class MapOutputTracker(conf:
SparkCo
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
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/20659
**[Test build #87613 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87613/testReport)**
for PR 20659 at commit
[`a5bb731`](https://github.com/apache/spark/commit/a5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
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/1005/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
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/20659
**[Test build #87613 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87613/testReport)**
for PR 20659 at commit
[`a5bb731`](https://github.com/apache/spark/commit/a
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87613/
Test FAILed.
---
Github user kiszk commented on the issue:
https://github.com/apache/spark/pull/20653
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20653
**[Test build #87614 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87614/testReport)**
for PR 20653 at commit
[`b5c8d56`](https://github.com/apache/spark/commit/b5
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20653
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/20653
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/1006/
Tes
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20651#discussion_r169919844
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/AllStagesPage.scala
---
@@ -172,6 +185,20 @@ private[ui] class AllStagesPage(parent: StagesTab)
ext
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20651
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/20651
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/1007/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20651
**[Test build #87615 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87615/testReport)**
for PR 20651 at commit
[`e04bf24`](https://github.com/apache/spark/commit/e0
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20659
**[Test build #87616 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87616/testReport)**
for PR 20659 at commit
[`80fd8a8`](https://github.com/apache/spark/commit/80
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
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/20659
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/1008/
Tes
Github user viirya commented on the issue:
https://github.com/apache/spark/pull/20648
From the codes, looks like there is an intention to have partial results
when failing to parse the documents. This patch makes the partial results. But
this should be considered as behavior change, a
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20653
**[Test build #87614 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87614/testReport)**
for PR 20653 at commit
[`b5c8d56`](https://github.com/apache/spark/commit/b
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20653
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87614/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20653
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/20659
**[Test build #87616 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87616/testReport)**
for PR 20659 at commit
[`80fd8a8`](https://github.com/apache/spark/commit/8
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87616/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20651
**[Test build #87615 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87615/testReport)**
for PR 20651 at commit
[`e04bf24`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20659
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/20651
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87615/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20651
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/20648
I was just double checking the current status for both CSV and JSON:
Seems CSV fills up the partial results with an exception (which is caught
by permissive mode with the corrupt record
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/20651
Jenkins, retest this please
---
-
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/20651
**[Test build #87617 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87617/testReport)**
for PR 20651 at commit
[`e04bf24`](https://github.com/apache/spark/commit/e0
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20651
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/1009/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20651
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/19788#discussion_r169967506
--- Diff: project/MimaExcludes.scala ---
@@ -1129,6 +1129,12 @@ object MimaExcludes {
ProblemFilters.exclude[InheritedNewAbstractMethodProblem]
Github user wangyum closed the pull request at:
https://github.com/apache/spark/pull/20659
---
-
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/20618
Was the result to add more changes to this PR or add them in another PR?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.a
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20652
lgtm
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r169983494
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/HeapMemoryAllocator.java
---
@@ -58,7 +58,8 @@ public MemoryBlock allocate(long size
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r169983548
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/HeapMemoryAllocator.java
---
@@ -70,7 +71,7 @@ public MemoryBlock allocate(long size
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r169983885
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/MemoryBlock.java ---
@@ -22,10 +22,9 @@
import org.apache.spark.unsafe.Platform;
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r169984899
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/MemoryBlock.java ---
@@ -45,38 +44,149 @@
*/
public static final int F
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r169985377
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/MemoryBlock.java ---
@@ -45,38 +44,149 @@
*/
public static final int F
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r169987267
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/MemoryBlock.java ---
@@ -45,38 +44,149 @@
*/
public static final int F
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r169987802
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/MemoryBlock.java ---
@@ -45,38 +44,149 @@
*/
public static final int F
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/20647#discussion_r169990652
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Relation.scala
---
@@ -107,17 +106,24 @@ case class DataSourc
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r169995800
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/MemoryBlock.java ---
@@ -45,38 +44,149 @@
*/
public static final int FREED
Github user jiangxb1987 commented on a diff in the pull request:
https://github.com/apache/spark/pull/20652#discussion_r169996395
--- Diff: core/src/test/scala/org/apache/spark/SecurityManagerSuite.scala
---
@@ -440,23 +440,41 @@ class SecurityManagerSuite extends SparkFunSuite wit
GitHub user gengliangwang opened a pull request:
https://github.com/apache/spark/pull/20660
[SPARK-23490][SQL]Check storage.locationUri with existing table in
CreateTable
## What changes were proposed in this pull request?
For CreateTable with Append mode, we should check i
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20660
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/1010/
Tes
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20660
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/20660
**[Test build #87618 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87618/testReport)**
for PR 20660 at commit
[`d72f56d`](https://github.com/apache/spark/commit/d7
Github user squito commented on the issue:
https://github.com/apache/spark/pull/20645
I agree it would be nicer to have this be a more general feature. I would
prefer an approach which didn't require a different configuration name, just as
its more to document & for users to keep tra
Github user gaborgsomogyi commented on a diff in the pull request:
https://github.com/apache/spark/pull/20652#discussion_r170003945
--- Diff: core/src/test/scala/org/apache/spark/SecurityManagerSuite.scala
---
@@ -440,23 +440,41 @@ class SecurityManagerSuite extends SparkFunSuite w
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20652
**[Test build #87619 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87619/testReport)**
for PR 20652 at commit
[`e01feef`](https://github.com/apache/spark/commit/e0
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20658#discussion_r170003647
--- Diff: python/pyspark/sql/catalog.py ---
@@ -137,6 +138,78 @@ def listColumns(self, tableName, dbName=None):
isBucket=jcolumn.isB
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20658#discussion_r170003442
--- Diff: python/pyspark/sql/catalog.py ---
@@ -28,7 +28,7 @@
Database = namedtuple("Database", "name description locationUri")
Table = namedtu
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20653#discussion_r170025695
--- Diff:
sql/core/src/test/scala/org/apache/spark/sql/execution/datasources/FileSourceStrategySuite.scala
---
@@ -491,6 +491,22 @@ class FileSourceStra
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20624#discussion_r170031873
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -387,6 +390,101 @@ case class CatalogStatistics(
Github user gatorsmile commented on a diff in the pull request:
https://github.com/apache/spark/pull/20624#discussion_r170028784
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/catalog/interface.scala
---
@@ -387,6 +390,101 @@ case class CatalogStatistics(
Github user gaborgsomogyi commented on the issue:
https://github.com/apache/spark/pull/20362
ping @srowen
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20651
**[Test build #87617 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87617/testReport)**
for PR 20651 at commit
[`e04bf24`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20651
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87617/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20651
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/20362
**[Test build #4126 has
started](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4126/testReport)**
for PR 20362 at commit
[`acfb092`](https://github.com/apache/spark/commit/a
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/20362#discussion_r170047180
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/recommendation/ALSSuite.scala ---
@@ -693,7 +766,7 @@ class ALSSuite
val data = ratings.toDF
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/20362#discussion_r170046857
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/recommendation/ALSSuite.scala ---
@@ -413,34 +411,36 @@ class ALSSuite
.setSeed(0)
val
Github user srowen commented on a diff in the pull request:
https://github.com/apache/spark/pull/20362#discussion_r170046788
--- Diff:
mllib/src/test/scala/org/apache/spark/ml/recommendation/ALSSuite.scala ---
@@ -586,6 +586,68 @@ class ALSSuite
allModelParamSettings, ch
Github user jose-torres commented on the issue:
https://github.com/apache/spark/pull/20622
The difference in ContinuousExecution is that the thread isn't doing any
metadata work like looking for new batches - it's either running the Spark job
or cleaning up after finishing it.
Github user lonehacker commented on the issue:
https://github.com/apache/spark/pull/10942
@gatorsmile Can you confirm if this feature is active in current master? It
seems like this code was removed in [SPARK-14535][SQL] Remove buildInternalScan
from FileFormat
---
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/20645
I like the `ConfigBuilder` approach. That would make this much more useful.
I'll add an implementation like that.
I think append option syntax would be confusing for users and
administrators
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20622
**[Test build #87620 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87620/testReport)**
for PR 20622 at commit
[`0e5e52f`](https://github.com/apache/spark/commit/0e
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20660
**[Test build #87618 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87618/testReport)**
for PR 20660 at commit
[`d72f56d`](https://github.com/apache/spark/commit/d
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20660
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/20660
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87618/
Test PASSed.
---
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20651#discussion_r170057919
--- Diff: core/src/main/scala/org/apache/spark/ui/jobs/AllStagesPage.scala
---
@@ -172,6 +185,20 @@ private[ui] class AllStagesPage(parent: StagesTab)
exten
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20657
retest this please
---
-
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/20657
Known flaky (SPARK-23458).
---
-
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/20651
LGTM, merging to master / 2.3.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/20651
Merged to master, but there were conflicts in 2.3; please open a separate
PR for the backport.
---
-
To unsubscribe, e-mail: revi
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20657
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/20657
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/1011/
Tes
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20657
**[Test build #87621 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87621/testReport)**
for PR 20657 at commit
[`2c3448d`](https://github.com/apache/spark/commit/2c
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/20651
---
-
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/20362
**[Test build #4126 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/NewSparkPullRequestBuilder/4126/testReport)**
for PR 20362 at commit
[`acfb092`](https://github.com/apache/spark/commit/
Github user kiszk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19222#discussion_r170068231
--- Diff:
common/unsafe/src/main/java/org/apache/spark/unsafe/memory/MemoryBlock.java ---
@@ -45,38 +44,149 @@
*/
public static final int FREED
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/20652
**[Test build #87619 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/87619/testReport)**
for PR 20652 at commit
[`e01feef`](https://github.com/apache/spark/commit/e
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20652
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/87619/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/20652
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional comma
1 - 100 of 215 matches
Mail list logo