Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19211
Following assert is failing in some cases, let me fix it and send a follow
up PR:
```
val listeners = sc.listenerBus.findListenersByClass[SaveExecutorInfo]
Github user dongjoon-hyun commented on the issue:
https://github.com/apache/spark/pull/19211
Hi, @vanzin and @cloud-fan .
This seems to break the master branch. Could you take a look this?
-
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19211
thanks, merging to master!
---
-
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/19211
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/19211
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81940/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81940 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81940/testReport)**
for PR 19211 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19211
I don't really have a readily available cluster that can easily put that
much pressure on the listener bus; I tried with the one I have and monitored
the listener bus metrics, and found pretty much
Github user squito commented on the issue:
https://github.com/apache/spark/pull/19211
lgtm
do you know how much better this makes it? Eg., if we had an existing case
where things go haywire in dynamic allocation because of this -- we could see
if after this change, the only
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81940 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81940/testReport)**
for PR 19211 at commit
Github user cloud-fan commented on the issue:
https://github.com/apache/spark/pull/19211
LGTM except some minor comments
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81895/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
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/19211
**[Test build #81895 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81895/testReport)**
for PR 19211 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81895 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81895/testReport)**
for PR 19211 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81884/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
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/19211
**[Test build #81884 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81884/testReport)**
for PR 19211 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81884 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81884/testReport)**
for PR 19211 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
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/19211
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81826/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81826 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81826/testReport)**
for PR 19211 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81826 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81826/testReport)**
for PR 19211 at commit
Github user ajbozarth commented on the issue:
https://github.com/apache/spark/pull/19211
I did a read-through of your code and didn't find any glaring issues, but
I'd like to go through it a bit more before passing judgment. Overall a nice
new feature though.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81795/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
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/19211
**[Test build #81795 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81795/testReport)**
for PR 19211 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81795 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81795/testReport)**
for PR 19211 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19211
I think this is ready now, pinging some people: @squito @JoshRosen
@cloud-fan
---
-
To unsubscribe, e-mail:
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
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/19211
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81744/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81744 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81744/testReport)**
for PR 19211 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81744 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81744/testReport)**
for PR 19211 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81728/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
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/19211
**[Test build #81728 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81728/testReport)**
for PR 19211 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81728 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81728/testReport)**
for PR 19211 at commit
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/19211
retest this please
---
-
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/19211
(Nevermind the test failures, I killed the obsolete builds.)
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
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/19211
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/19211
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/19211
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81724/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81726/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81725/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81726 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81726/testReport)**
for PR 19211 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81725 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81725/testReport)**
for PR 19211 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81724 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81724/testReport)**
for PR 19211 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/19211
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/19211
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/81693/
Test PASSed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81693 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81693/testReport)**
for PR 19211 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/19211
**[Test build #81693 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/81693/testReport)**
for PR 19211 at commit
51 matches
Mail list logo
