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.

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

Add a @pub kwarg to allow specifying a "startup response message" #189

Closed
wants to merge 4 commits into from

Conversation

goodboy
Copy link
Owner

@goodboy goodboy commented Jan 24, 2021

Add a way to specify an initial "startup response message" when first connecting to a @tractor.msg.pub streaming function.

@msg.pub is a provisional and undocumented api for a small internal dynamic pub-sub system.

This will likely need some tests and thought about if it's superfluous.
The main use case we had in piker was for some kind of synchronization/handshake for order management daemons connecting to data feeds on spawn.

@goodboy goodboy force-pushed the pub_connect_msg branch 4 times, most recently from e9fc328 to 7647c56 Compare February 25, 2021 14:53
goodboy added 4 commits April 27, 2021 11:35
Add a sync method that can be used to cancel the current actor from
a synchronous context. This is useful in debugging situations where
sync debugger code may need to kill the process tree.

Also, make the internal "lifetime stack" a global var; easier to manage
from client code that may was to add callbacks prior to the actor
runtime being fully setup.
The std lib's `pdb` internals override SIGINT handling whenever one
enters the debugger repl. Force a handler that kills the tree if SIGINT
is triggered from the root actor, otherwise igore it since supervised
children should be managed already. This resolves an issue with guest
mode where `pdb` causes SIGINTs to be swallowed resulting in the host
loop never terminating the process tree.
@goodboy
Copy link
Owner Author

goodboy commented Nov 9, 2021

Replaced by #258

@goodboy goodboy closed this Nov 9, 2021
@goodboy goodboy deleted the pub_connect_msg branch November 9, 2021 02:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant