Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22450
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19045
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22460
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96288/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22460
**[Test build #96288 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96288/testReport)**
for PR 22460 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19045
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22450
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22460
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22460
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22460
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22465
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22465
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19045
**[Test build #96290 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96290/testReport)**
for PR 19045 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22450
**[Test build #96289 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96289/testReport)**
for PR 22450 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22469
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22460
**[Test build #96288 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96288/testReport)**
for PR 22460 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22469
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22469
**[Test build #96286 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96286/testReport)**
for PR 22469 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22465
**[Test build #96287 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96287/testReport)**
for PR 22465 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22450
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/22010#discussion_r218946996
--- Diff: core/src/main/scala/org/apache/spark/rdd/MapPartitionsRDD.scala
---
@@ -35,16 +35,24 @@ import org.apache.spark.{Partition, TaskContext}
*
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22469
thank you for pointing this out @cloud-fan
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mridulm commented on a diff in the pull request:
https://github.com/apache/spark/pull/22010#discussion_r218946895
--- Diff: core/src/main/scala/org/apache/spark/rdd/RDD.scala ---
@@ -396,7 +396,26 @@ abstract class RDD[T: ClassTag](
* Return a new RDD containing
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22450
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22450
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96254/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22450
**[Test build #96254 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96254/testReport)**
for PR 22450 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22465
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22470
@cloud-fan I think the main problem about this (and it is the reason why I
haven't proposed it) is that the range of operations supported would be
smaller, so we may forbid operations which now
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22460
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96270/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22460
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22460
**[Test build #96270 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96270/testReport)**
for PR 22460 at commit
Github user rezasafi commented on a diff in the pull request:
https://github.com/apache/spark/pull/22325#discussion_r218944816
--- Diff:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
---
@@ -444,36 +444,34 @@ final class
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22325
**[Test build #96285 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96285/testReport)**
for PR 22325 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22450
@cloud-fan I checked the other operations and they have no issue with
negative scale. This is the reason why this fix is only for Divide: it is the
only operation which wasn't dealing it properly.
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/22472
Probably revert itself can be done directly in the master branch or it has
to be done in a PR like this?
---
-
To unsubscribe,
Github user redsanket commented on a diff in the pull request:
https://github.com/apache/spark/pull/22173#discussion_r218942035
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/TransportContext.java
---
@@ -77,17 +82,54 @@
private static final
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22473
**[Test build #96284 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96284/testReport)**
for PR 22473 at commit
Github user zsxwing commented on a diff in the pull request:
https://github.com/apache/spark/pull/22473#discussion_r218941326
--- Diff: core/src/main/scala/org/apache/spark/executor/Executor.scala ---
@@ -799,7 +799,8 @@ private[spark] class Executor(
if (taskRunner.task
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/22413
@dongjoon-hyun Here is the PR for 2.4:
https://github.com/apache/spark/pull/22474 . I will check does it have
conflicts in 2.3 and if so, I will backport it to 2.3
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22173
**[Test build #96283 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96283/testReport)**
for PR 22173 at commit
Github user zsxwing commented on the issue:
https://github.com/apache/spark/pull/22473
add to whitelist
---
-
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/22474
**[Test build #96282 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96282/testReport)**
for PR 22474 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22474
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/22474
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/22474
[SPARK-25425][SQL][BACKPORT-2.4] Extra options should override session
options in DataSource V2
## What changes were proposed in this pull request?
In the PR, I propose overriding session
Github user redsanket commented on a diff in the pull request:
https://github.com/apache/spark/pull/22173#discussion_r218939333
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/TransportContext.java
---
@@ -98,21 +98,32 @@ public TransportContext(
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22472
The reverting PR title is usually `Revert "[SPARK-23173][SQL] rename
spark.sql.fromJsonForceNullableSchema"`
---
-
To
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22470
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96271/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22470
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22470
**[Test build #96271 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96271/testReport)**
for PR 22470 at commit
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/22472
> Let's not mix the reverting and adding a deprecation note
ok. I will revert the last commit from the PR.
> Also, SPARK-25384 is about Removing
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/22448
> Do you know the reason for it ?
It is better to ask @HyukjinKwon
> Seems like we may need to refactor to see if we can avoid duplicating
findTightestCommonType here ? Can we
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/21688#discussion_r218931652
--- Diff: core/src/main/resources/org/apache/spark/ui/static/stagepage.js
---
@@ -0,0 +1,926 @@
+/*
+ * Licensed to the Apache Software
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22305
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22305
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96260/
Test PASSed.
---
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/21688#discussion_r218931328
--- Diff: core/src/main/resources/org/apache/spark/ui/static/stagepage.js
---
@@ -0,0 +1,926 @@
+/*
+ * Licensed to the Apache Software
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22305
**[Test build #96260 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96260/testReport)**
for PR 22305 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22325
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96265/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22325
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22325
**[Test build #96265 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96265/testReport)**
for PR 22325 at commit
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22448
@MaxGekk I was looking at CSVInferSchema. It seems like there is a copy of
`findTightestCommonType` in this file ? Do you know the reason for it ? Seems
like we may need to refactor to see if
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22447
**[Test build #96280 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96280/testReport)**
for PR 22447 at commit
Github user redsanket commented on a diff in the pull request:
https://github.com/apache/spark/pull/22173#discussion_r218930135
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/TransportContext.java
---
@@ -77,17 +82,43 @@
private static final
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22173
**[Test build #96281 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96281/testReport)**
for PR 22173 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22447
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22447
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22325
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user maryannxue commented on the issue:
https://github.com/apache/spark/pull/22447
retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22325
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96269/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22325
**[Test build #96269 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96269/testReport)**
for PR 22325 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22447
Merged build finished. Test FAILed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22447
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96272/
Test FAILed.
---
Github user rdblue commented on the issue:
https://github.com/apache/spark/pull/22413
Thanks @MaxGekk, sorry for the original omission!
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22447
**[Test build #96272 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96272/testReport)**
for PR 22447 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22413
Thank you so much, @MaxGekk . It would be great if we can have that Spark
2.4.0 RC2.
---
-
To unsubscribe, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22472
cc @gatorsmile , too.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/22173#discussion_r218926387
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/TransportContext.java
---
@@ -77,17 +82,43 @@
private static final
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22472
Let's not mix the reverting and adding a deprecation note, @MaxGekk .
Also, `SPARK-25384` is about `Removing
spark.sql.fromJsonForceNullableSchema`, not `Deprecating` at all.
---
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/22173#discussion_r218925291
--- Diff:
common/network-common/src/test/java/org/apache/spark/network/ChunkFetchRequestHandlerSuite.java
---
@@ -0,0 +1,102 @@
+/*
+ * Licensed
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22473
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user dilipbiswal commented on the issue:
https://github.com/apache/spark/pull/22470
@cloud-fan Could you please check CSVinferSchema::tryParseDecimal() ? There
is a condition to check negative scale.
---
-
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/22173#discussion_r218924768
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/server/ChunkFetchRequestHandler.java
---
@@ -0,0 +1,136 @@
+/*
+ *
Github user tgravescs commented on a diff in the pull request:
https://github.com/apache/spark/pull/22173#discussion_r218924588
--- Diff:
common/network-common/src/main/java/org/apache/spark/network/TransportContext.java
---
@@ -173,5 +213,14 @@ private TransportChannelHandler
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22473
**[Test build #96279 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96279/testReport)**
for PR 22473 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22473
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user mukulmurthy commented on the issue:
https://github.com/apache/spark/pull/22473
@zsxwing for review
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user mukulmurthy opened a pull request:
https://github.com/apache/spark/pull/22473
[SPARK-25449][CORE] Heartbeat shouldn't include accumulators for zero
metrics
## What changes were proposed in this pull request?
Heartbeat shouldn't include accumulators for zero
Github user MaxGekk commented on the issue:
https://github.com/apache/spark/pull/22413
> There is some conflict. Please make backporting PRs to 2.4 and 2.3.
> BTW, @MaxGekk . Could you send a backport PR to branch-2.4?
Sure, I will do that.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22462
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22462
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96266/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22468
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96262/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22468
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22462
**[Test build #96266 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96266/testReport)**
for PR 22462 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22472
**[Test build #96278 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96278/testReport)**
for PR 22472 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22468
**[Test build #96262 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96262/testReport)**
for PR 22468 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22472
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/22472
Can one of the admins verify this patch?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
GitHub user MaxGekk opened a pull request:
https://github.com/apache/spark/pull/22472
[SPARK-23173][SPARK-25384][SQL] Reverting of
spark.sql.fromJsonForceNullableSchema
## What changes were proposed in this pull request?
The flag has been released already in 2.3.x.
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22010#discussion_r218918701
--- Diff: core/src/main/scala/org/apache/spark/rdd/RDD.scala ---
@@ -396,7 +396,26 @@ abstract class RDD[T: ClassTag](
* Return a new RDD containing
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22010#discussion_r218917483
--- Diff: core/src/main/scala/org/apache/spark/rdd/MapPartitionsRDD.scala
---
@@ -35,16 +35,24 @@ import org.apache.spark.{Partition, TaskContext}
*
Github user mn-mikke commented on a diff in the pull request:
https://github.com/apache/spark/pull/20858#discussion_r218917303
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/collectionOperations.scala
---
@@ -665,3 +667,219 @@ case class
401 - 500 of 919 matches
Mail list logo