Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[ASIO] Make actors better and more wonderful. #183

Merged
merged 2 commits into from
Apr 1, 2015
Merged

[ASIO] Make actors better and more wonderful. #183

merged 2 commits into from
Apr 1, 2015

Conversation

kobolog
Copy link
Member

@kobolog kobolog commented Mar 30, 2015

Thread-safe actor_t, finally! Also, made the header public for use in plugins, specifically cluster managers.

@kobolog
Copy link
Member Author

kobolog commented Mar 30, 2015

@3Hren @antmat: WTB review!

@kobolog kobolog changed the title [ASIO] Actor thread safety. [ASIO] Actor thread safety! Mar 30, 2015
@kobolog kobolog force-pushed the kobolog/actor-threads branch 4 times, most recently from 4ac9022 to f846a65 Compare March 31, 2015 00:51
@kobolog
Copy link
Member Author

kobolog commented Mar 31, 2015

Note that calls to run() and terminate() are synchronized by the Context, so the only thing we care about here is concurrent calls to endpoint() and terminate(), for example.

@kobolog kobolog force-pushed the kobolog/actor-threads branch 2 times, most recently from 6869158 to f972463 Compare March 31, 2015 02:20
@kobolog kobolog changed the title [ASIO] Actor thread safety! [ASIO] Make actors better and more wonderful. Apr 1, 2015
@antmat
Copy link
Contributor

antmat commented Apr 1, 2015

Looks good

kobolog pushed a commit that referenced this pull request Apr 1, 2015
Merge pull request #183 from cocaine/kobolog/actor-threads
8738d5f 
[ASIO] Make actors better and more wonderful.
@kobolog kobolog merged commit 8738d5f into master Apr 1, 2015
@kobolog kobolog deleted the kobolog/actor-threads branch April 1, 2015 15:40
antmat added a commit to antmat/cocaine-core that referenced this pull request Jun 19, 2017
@antmat
fix: remove quadratic complexity from encoder (cocaine#183)
49fbc7c
3Hren pushed a commit that referenced this pull request Aug 18, 2017
@antmat @3Hren
fix: remove quadratic complexity from encoder (#183)
4699297
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@kobolog @antmat