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

Fix/check if pid is alive on assignments revoked #44

Merged
merged 4 commits into from
Jan 13, 2021
Merged

Fix/check if pid is alive on assignments revoked #44

merged 4 commits into from
Jan 13, 2021

Conversation

amacciola
Copy link
Contributor

Prevents Genserver crash error if the PID does not exist because the pipeline is being shutdown

@amacciola amacciola mentioned this pull request Jan 13, 2021
Closed
@josevalim josevalim merged commit c652048 into dashbitco:master Jan 13, 2021
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

@amacciola
Copy link
Contributor Author

@v0idpwn @josevalim link to new PR #45
This one that was merged still had an case were it would break. I handled it in the new PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@v0idpwn v0idpwn v0idpwn approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@amacciola @josevalim @v0idpwn