Github user HyukjinKwon commented on the issue:
https://github.com/apache/spark/pull/16291
what's the current status of this PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user CodingCat commented on the issue:
https://github.com/apache/spark/pull/16291
what's the current status of this PR?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/16291
@tgravescs - I have not gotten time to work on this. Hopefully will get to
it sometime next week.
---
If your project is set up for it, you can reply to this email and have your
reply appear on
Github user tgravescs commented on the issue:
https://github.com/apache/spark/pull/16291
@sitalkedia are you still working on this?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/16291
Thanks guys for your input. I have made changes to support grouping of
listeners to preserve the existing synchronization behavior. Currently, the
`ExecutionAllocationManager`,
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70660/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70660 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70660/testReport)**
for PR 16291 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70660 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70660/testReport)**
for PR 16291 at commit
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/16291
Jenkins retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70656/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70656 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70656/testReport)**
for PR 16291 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70652/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70652 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70652/testReport)**
for PR 16291 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70652 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70652/testReport)**
for PR 16291 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70651 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70651/testReport)**
for PR 16291 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70651/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70651 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70651/testReport)**
for PR 16291 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70650/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70650 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70650/testReport)**
for PR 16291 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70650 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70650/testReport)**
for PR 16291 at commit
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/16291
jenkins test this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70649/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70649 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70649/testReport)**
for PR 16291 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70649 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70649/testReport)**
for PR 16291 at commit
Github user mridulm commented on the issue:
https://github.com/apache/spark/pull/16291
I agree with @markhamstra and @vanzin - having ability to tag listeners
into groups (default = spark listener group) and preserving current
synchronized behavior within group would be ensure
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/16291
> I think that will be an over-kill. Having a separate thread per listener
simplifies the design.
I don't think it's overkill, and it wouldn't really be that much of a
change from your
Github user markhamstra commented on the issue:
https://github.com/apache/spark/pull/16291
> Each listener should in theory could work independent of each other and
we should only guarantee ordered processing of the events within a listener.
If we were starting from nothing,
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/16291
@markhamstra, @vanzin thanks for your comments!
>> so it is entirely possible. for example, for one ListenerEventExecutor
to process a task end event for a particular task before another
Github user vanzin commented on the issue:
https://github.com/apache/spark/pull/16291
I'm not sure whether that change would be a problem, but I wonder if a
system where listeners can "opt in" to this behavior wouldn't be better. e.g.,
`EventLoggingListener` can slow down the bus,
Github user markhamstra commented on the issue:
https://github.com/apache/spark/pull/16291
> You are right, that's why the executor service is single threaded which
guarantees ordered processing of the events per listener.
But this is still substantially different from the
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/16291
This change is ready for review @rxin, @zsxwing, @kayousterhout
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70360/
Test PASSed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test PASSed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70360 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70360/testReport)**
for PR 16291 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70360 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70360/testReport)**
for PR 16291 at commit
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/16291
Jenkins retest this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70345/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70345 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70345/testReport)**
for PR 16291 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70325/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70325 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70325/testReport)**
for PR 16291 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70325 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70325/testReport)**
for PR 16291 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70322/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70322 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70322/testReport)**
for PR 16291 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70322 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70322/testReport)**
for PR 16291 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70298/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70299/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70299 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70299/testReport)**
for PR 16291 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70298 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70298/testReport)**
for PR 16291 at commit
Github user sitalkedia commented on the issue:
https://github.com/apache/spark/pull/16291
Jenkins test this please.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70290/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70290 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70290/testReport)**
for PR 16291 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70290 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70290/testReport)**
for PR 16291 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70289/
Test FAILed.
---
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70289 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70289/testReport)**
for PR 16291 at commit
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70289 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70289/testReport)**
for PR 16291 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Merged build finished. Test FAILed.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70287 has
finished](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70287/testReport)**
for PR 16291 at commit
Github user AmplabJenkins commented on the issue:
https://github.com/apache/spark/pull/16291
Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/70287/
Test FAILed.
---
Github user SparkQA commented on the issue:
https://github.com/apache/spark/pull/16291
**[Test build #70287 has
started](https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/70287/testReport)**
for PR 16291 at commit
72 matches
Mail list logo