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

NetworkLayer.listen: don't "die" when node backend fails #149

Merged
merged 1 commit into from
Apr 4, 2019

Conversation

rvl
Copy link
Contributor

@rvl rvl commented Apr 4, 2019

Part of investigations into #100.

Overview

"die" will exit the process, which might not be what we want in all cases. Throw an exception instead, which will also result in the process exiting, but could be caught if needed.

@rvl rvl self-assigned this Apr 4, 2019
"die" will exit the process, which might not be what we want in all
cases. Throw an exception instead, which will also result in the
process exiting, but could be caught if needed.
@rvl rvl force-pushed the rvl/listen-no-die branch from 366200b to ea87377 Compare April 4, 2019 06:07
@KtorZ
Copy link
Member

KtorZ commented Apr 4, 2019

🎉 Yeay!
I noticed the other day while restoring stuff against mainnet and forgot to mention in the weekly. Thanks for fixing!

@KtorZ
Copy link
Member

KtorZ commented Apr 4, 2019

In the long-run, I wonder if we should just handle some of those with a log message, perhaps with a small re-connection delay of something like this. This could also just be done by an external service supervising our application in production; following a bit Erlang strategy: crash early.

@KtorZ KtorZ merged commit 7f76504 into master Apr 4, 2019
@KtorZ KtorZ deleted the rvl/listen-no-die branch April 4, 2019 06:36
rvl added a commit that referenced this pull request Sep 18, 2020
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.

2 participants