-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[Distributed] Offer LocalTestingDistributedActorSystem #41756
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Distributed] Offer LocalTestingDistributedActorSystem #41756
Conversation
@ktoso Please review |
stdlib/public/Distributed/LocalTestingDistributedActorSystem.swift
Outdated
Show resolved
Hide resolved
stdlib/public/Distributed/LocalTestingDistributedActorSystem.swift
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking good :) Need to figure out what locks we can use here to make it threadsafe...
524e054
to
3f3c9c2
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! We need to make it thread safe and we're good here :)
d3d98e1
to
547bf8e
Compare
547bf8e
to
ec2fa76
Compare
#41804 was fixed so we can now use the |
Add local-only actor system `LocalTestingDistributedActorSystem` to make code and tutorial development easier. rdar://89580224
1e70364
to
b001d5b
Compare
@swift-ci please smoke test |
Not sure why I can't PR into your branch, but here's a minor patch that'll make us able ot merge this ktoso@a49310b Thanks a lot @yim-lee ! |
// UNSUPPORTED: use_os_stdlib | ||
// UNSUPPORTED: back_deployment_runtime | ||
|
||
import _Distributed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sadly gotta test this all again since module name change
@swift-ci please smoke test and merge |
Add local-only actor system
LocalTestingDistributedActorSystem
to make code and tutorial development easier.rdar://89580224