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

reintroduce signal handling #287

Open
jhellerstein opened this issue Jan 21, 2013 · 2 comments
Open

reintroduce signal handling #287

jhellerstein opened this issue Jan 21, 2013 · 2 comments
Assignees

Comments

@jhellerstein
Copy link
Member

We used to have a buggy signal handling solution, which was removed in commit 032bf1d. It would be good to have a similar but working solution that addresses the problems discussed in that commit.

@ghost ghost assigned jhellerstein Jan 21, 2013
@neilconway
Copy link
Member

Sure, happy to chat about a sane way to provide support for it. Roughly speaking, I think it should be safe to:

  1. set global variables in the signal handler
  2. periodically poll those variables in the signal check thread
  3. have that thread enqueue events to be consumed by the active Bud instances, which could then turn them into tuples; similarly to how inbound network events are handled

@jhellerstein
Copy link
Member Author

Yes, making it work isomorphically to network events (and periodics) would be best.

J

On Jan 21, 2013, at 3:31 PM, Neil Conway [email protected] wrote:

Sure, happy to chat about a sane way to provide support for it. Roughly speaking, I think it should be safe to:

set global variables in the signal handler
periodically poll those variables in the signal check thread
have that thread enqueue events to be consumed by the active Bud instances, which could then turn them into tuples; similarly to how inbound network events are handled

Reply to this email directly or view it on GitHub.

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

No branches or pull requests

2 participants