Github user rxin commented on the issue:
https://github.com/apache/spark/pull/22472
im ok either way
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22472
@MaxGekk . It's reverted from `master` and `branch-2.4`. Could you close
this PR? Thanks!
---
-
To unsubscribe, e-mail:
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/22376
The 3 failing tests reported in Jenkins were fixed in
76514a015168de8d8b54b3abf6b835050eefd8c2 and are unrelated to this change.
---
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/22376
That build failure _seems_ to be a host issue, but lets kick off a retest
quickly anyways.
Jenkins retest this please.
---
Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22449
---
-
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/22376
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/3266/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22471
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96277/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22471
**[Test build #96277 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96277/testReport)**
for PR 22471 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22471
Merged build finished. Test PASSed.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/22475
done
---
-
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/22192
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96276/
Test PASSed.
---
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/22476
done
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22465#discussion_r218998728
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala ---
@@ -1561,6 +1561,13 @@ object SQLConf {
"are performed
Github user brkyvz commented on the issue:
https://github.com/apache/spark/pull/22478
cc @zsxwing @jose-torres
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
GitHub user brkyvz opened a pull request:
https://github.com/apache/spark/pull/22478
[SPARK-25472] Don't have legitimate stops of streams cause stream exceptions
## What changes were proposed in this pull request?
Legitimate stops of streams may actually cause an exception
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/19045#discussion_r219000926
--- Diff:
core/src/main/scala/org/apache/spark/scheduler/ExecutorLossReason.scala ---
@@ -58,3 +58,11 @@ private [spark] object LossReasonPending extends
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22479
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/22479
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/22479
**[Test build #96311 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96311/testReport)**
for PR 22479 at commit
Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/22479
cc @BryanCutler
---
-
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/22476
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/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/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/22173
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96256/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22173
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/22173
**[Test build #96256 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96256/testReport)**
for PR 22173 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22138
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/22138
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96295/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22138
**[Test build #96295 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96295/testReport)**
for PR 22138 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21404
@tooptoop4 . Since this is a new feature, it's now targeting for Apache
Spark 2.5 since `branch-2.4` is already cut. In addition, this will not be
allowed for backporting `branch-2.4`. Is it
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22462
**[Test build #96275 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96275/testReport)**
for PR 22462 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22476
**[Test build #96297 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96297/testReport)**
for PR 22476 at commit
Github user mgaido91 commented on the issue:
https://github.com/apache/spark/pull/22470
@dilipbiswal yes, I definitely think that in general we should get to
forbid negative scales. I only thought that this should be done in 3.0 rather
than now. And for now the safest option to me
Github user MaxGekk closed the pull request at:
https://github.com/apache/spark/pull/22472
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/22376
Jenkins, ok to test.
---
-
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/22376
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/3266/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22472
**[Test build #96278 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96278/testReport)**
for PR 22472 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22376
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/22376
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/22376
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/3267/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22477
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/22477
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
Github user tooptoop4 commented on the issue:
https://github.com/apache/spark/pull/21404
@dongjoon-hyun I was planning to do my own custom build by cherrypicking
your PR if you had it available
---
-
To
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22474
Thank you, @MaxGekk . Merged to `branch-2.4`.
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/21404
That would be helpful for us in the end. But, sorry for now. I'm currently
not planning it open soon because the next release (Apache Spark 2.5 or 3.0)
will next year.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22450
**[Test build #96289 has
finished](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/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/22450
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96289/
Test PASSed.
---
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/22475
LGTM
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
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/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 SparkQA commented on the issue:
https://github.com/apache/spark/pull/22469
**[Test build #96286 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96286/testReport)**
for PR 22469 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 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):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96286/
Test PASSed.
---
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22413
I see. Thank you, @MaxGekk !
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/22471
@ueshin can you review?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user tooptoop4 commented on the issue:
https://github.com/apache/spark/pull/21404
bump
---
-
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/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/96275/
Test PASSed.
---
Github user holdenk commented on the issue:
https://github.com/apache/spark/pull/22376
Jenkins retest this please
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user rxin commented on the issue:
https://github.com/apache/spark/pull/21169
i'm actually not sure if we should do this, given impala treats timestamp
as timestamp without timezone, whereas spark treats it as a utc timestamp (with
timezone). these functions are super confusing
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22473
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/22473
**[Test build #96301 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96301/testReport)**
for PR 22473 at commit
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22475
Could you add `[FOLLOWUP]` tag to the title, too?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22325#discussion_r218990774
--- Diff:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
---
@@ -444,36 +445,36 @@ final class ShuffleBlockFetcherIterator(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22376
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-make-spark-distribution-unified/3267/
---
Github user vanzin commented on a diff in the pull request:
https://github.com/apache/spark/pull/22325#discussion_r218991156
--- Diff:
core/src/main/scala/org/apache/spark/storage/ShuffleBlockFetcherIterator.scala
---
@@ -444,36 +445,36 @@ final class ShuffleBlockFetcherIterator(
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22173
**[Test build #96307 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96307/testReport)**
for PR 22173 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22474
**[Test build #96282 has
finished](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
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/22474
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96282/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22173
**[Test build #96274 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96274/testReport)**
for PR 22173 at commit
Github user BryanCutler commented on the issue:
https://github.com/apache/spark/pull/22477
cc @HyukjinKwon
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail:
Github user maropu commented on the issue:
https://github.com/apache/spark/pull/22461
ok, thanks! Anyway, LGTM
---
-
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/22473
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/22473
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96279/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22473
**[Test build #96279 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96279/testReport)**
for PR 22473 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22460
**[Test build #96310 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96310/testReport)**
for PR 22460 at commit
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 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/22460
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/22173
**[Test build #96294 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96294/testReport)**
for PR 22173 at commit
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/22173
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96291/
Test FAILed.
---
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 dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/22472
IIUC, reverting PRs also need other committers' LGTM in general.
@rxin , @gatorsmile . If allowed, I can revert this in `master` and
`branch-2.4` without explict PRs as @MaxGekk
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22476
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/22477
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/22477
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/22173
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96305/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/22192
**[Test build #96276 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96276/testReport)**
for PR 22192 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/22173
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/22192
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/22173
**[Test build #96305 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/96305/testReport)**
for PR 22173 at commit
101 - 200 of 919 matches
Mail list logo