Skip to content

[Distributed] inject transport.resign into deinits #38129

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

Merged
merged 2 commits into from
Jul 1, 2021

Conversation

ktoso
Copy link
Contributor

@ktoso ktoso commented Jun 28, 2021

Distributed actor deinits must resign their address.

@ktoso ktoso added the distributed Feature → concurrency: distributed actor label Jun 28, 2021
@ktoso ktoso marked this pull request as draft June 28, 2021 15:29
@ktoso ktoso force-pushed the wip-resignaddress-on-deinit branch from 82f4b3e to adea6d9 Compare June 29, 2021 09:42
@ktoso ktoso requested a review from slavapestov June 30, 2021 10:15
@ktoso ktoso force-pushed the wip-resignaddress-on-deinit branch from 2c7a2ea to 3e77eac Compare July 1, 2021 06:40
@ktoso ktoso marked this pull request as ready for review July 1, 2021 06:40
@ktoso
Copy link
Contributor Author

ktoso commented Jul 1, 2021

@swift-ci please smoke test

@ktoso ktoso force-pushed the wip-resignaddress-on-deinit branch from 044f859 to 04a2dd9 Compare July 1, 2021 09:17
@ktoso
Copy link
Contributor Author

ktoso commented Jul 1, 2021

@swift-ci please smoke test and merge

@swift-ci swift-ci merged commit 11fbdaa into swiftlang:main Jul 1, 2021
@ktoso ktoso deleted the wip-resignaddress-on-deinit branch July 1, 2021 11:51
@ktoso
Copy link
Contributor Author

ktoso commented Jul 5, 2021

resolves rdar://80005424

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
distributed Feature → concurrency: distributed actor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants