Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3699/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3699/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/3843/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
**[Test build #91545 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91545/testReport)**
for PR 21366 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
Test FAILed.
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/3842/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
**[Test build #91544 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91544/testReport)**
for PR 21366 at commit
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/21366
@foxish @liyinan926 @dvogelbacher ok, I think what we have here is closer
to what we want. I primarily addressed the situations outlined by the most
recent comments and observations made about the
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
**[Test build #91543 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91543/testReport)**
for PR 21366 at commit
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/21366
That's what should happen, but I think that isn't handled by this code. To
handle that I think the abstractions need to be a bit different from what we
have here. I'll draft up something. Hopefully
Github user foxish commented on the issue:
https://github.com/apache/spark/pull/21366
If we did miss the watch as you said, when the next poll comes along, that
should capture the fact that a pod that we thought existed doesn't exist
anymore in the API. So, that would be the level
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/21366
I think there's a single case here that could be missed.
If we delete a pod from the API server entirely, and neither the watch
receives a Pod with the deletion timestamp set OR the polling
Github user foxish commented on the issue:
https://github.com/apache/spark/pull/21366
Yes, watches + polling things is common in controllers. Shared informers
are a good stable abstractions they use so, they can avoid the polling in most
cases. However, for now, If we just use the
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/21366
Do most controllers use both Watches and polling? I'm wondering if we can
just poll very frequently and not have the watch.
---
Github user foxish commented on the issue:
https://github.com/apache/spark/pull/21366
Sorry, not yet. I explained to @mccheah what I meant about us preferring
the level triggered approach and always being able to rebuild state even if
some events are lost and resyncing completely on
Github user dvogelbacher commented on the issue:
https://github.com/apache/spark/pull/21366
@foxish did you have a chance to take a look?
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91464/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
**[Test build #91464 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91464/testReport)**
for PR 21366 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91463/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
**[Test build #91463 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91463/testReport)**
for PR 21366 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91466/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
**[Test build #91466 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91466/testReport)**
for PR 21366 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3664/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3664/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
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/3805/
Github user foxish commented on the issue:
https://github.com/apache/spark/pull/21366
I was planning to do one last pass over it today/tomorrow. I can merge
after that.
---
-
To unsubscribe, e-mail:
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/21366
@foxish or @felixcheung, there have been a few +1s - did either of you want
to make the call to merge here? I can also merge.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3663/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3663/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
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/3804/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
Test FAILed.
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/3803/
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
**[Test build #91466 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91466/testReport)**
for PR 21366 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
**[Test build #91464 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91464/testReport)**
for PR 21366 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
Test FAILed.
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/3802/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
**[Test build #91463 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91463/testReport)**
for PR 21366 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91319/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
**[Test build #91319 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91319/testReport)**
for PR 21366 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3580/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/3713/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3580/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
**[Test build #91319 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91319/testReport)**
for PR 21366 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91281/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
**[Test build #91281 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91281/testReport)**
for PR 21366 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3555/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3555/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/3688/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
**[Test build #91281 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91281/testReport)**
for PR 21366 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91273/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
**[Test build #91273 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91273/testReport)**
for PR 21366 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3549/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3549/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
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/3682/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
**[Test build #91273 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91273/testReport)**
for PR 21366 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91164/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
**[Test build #91164 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91164/testReport)**
for PR 21366 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91165/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
**[Test build #91165 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91165/testReport)**
for PR 21366 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91159/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
**[Test build #91159 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91159/testReport)**
for PR 21366 at commit
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/21366
@eje or @felixcheung would either of you like to add further comment and/or
sign off here?
---
-
To unsubscribe, e-mail:
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3468/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3468/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
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/3593/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3467/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
**[Test build #91165 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91165/testReport)**
for PR 21366 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3467/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
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/3592/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
**[Test build #91164 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91164/testReport)**
for PR 21366 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3464/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3464/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/3589/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test status success
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3463/
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
Kubernetes integration test starting
URL:
https://amplab.cs.berkeley.edu/jenkins/job/testing-k8s-prb-spark-integration/3463/
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
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/3588/
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/21366
**[Test build #91159 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91159/testReport)**
for PR 21366 at commit
Github user mccheah commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91134/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/21366
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/21366
**[Test build #91134 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/91134/testReport)**
for PR 21366 at commit
1 - 100 of 117 matches
Mail list logo