Skip to content
This repository has been archived by the owner on Dec 11, 2018. It is now read-only.

match against ok and close instead of crashing on close #79

Merged
merged 2 commits into from
Dec 22, 2015

Conversation

marianoguerra
Copy link
Contributor

No description provided.

@marianoguerra
Copy link
Contributor Author

PR for issue #77

2015-08-20 06:06:15 =ERROR REPORT====
Error in process <0.27879.32> on node '[email protected]' with exit value:
{[
{reason,{case_clause,{error,closed}}},
{mfa,{bullet_handler,info,3}},
{stacktrace,[{bullet_handler,reply_get_mode,3,[
{file,"src/bullet_handler.erl"},{line,201}]},
{bullet_handler,info,3,[{file,"src/bullet_handler.erl"},{line...
@marianoguerra
Copy link
Contributor Author

just added another match which I could see in the logs after the ones from the previous commit weren't there anymore, it seems to happen less frequently.

@essen
Copy link
Member

essen commented Aug 20, 2015

Cheers; I need to move this repository to ninenines, I'm not seeing it when I do work on tickets. Feel free to remind me if I don't before this week-end. :-)

@marianoguerra
Copy link
Contributor Author

bump! :)

@essen
Copy link
Member

essen commented Aug 31, 2015

OK I will try to work on it this week-end, remind me at that time please. Right now it's not possible.

@marianoguerra
Copy link
Contributor Author

bump (don't worry if you are busy, just reminding you as requested :)

@mokevnin
Copy link

mokevnin commented Oct 8, 2015

Bump), we have the same issue.

@essen essen added the Bug label Nov 12, 2015
@marianoguerra
Copy link
Contributor Author

just to add a note, this has been running in production for me since august without a problem.

@essen essen merged commit 8053fc2 into ninenines:master Dec 22, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants