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

connection timeout added #79

Merged
merged 1 commit into from Jun 1, 2017

Conversation

Projects
None yet
2 participants
@furusiyya
Collaborator

furusiyya commented May 31, 2017

#72 fixed!
[user.tcp] accept tcp [::]:5000: accept4: too many open files was due to the allowance of limited number of open file descriptors by the operating system. There was no deadline set for opened connections so most of the connections never get closed. In result, the number of opened connections gradually cross the maximum open file descriptors limit and cause panic.
Now with connection timeout = 72 second, number of opened connection will never reach the open file descriptor limit.

@furusiyya furusiyya changed the title from Connection timeout added to connections timeout added May 31, 2017

@furusiyya furusiyya changed the title from connections timeout added to connection timeout added May 31, 2017

@glaslos

This comment has been minimized.

Show comment
Hide comment
@glaslos

glaslos Jun 1, 2017

Member

Good works, looks solid. Will merge and test it in production.

Member

glaslos commented Jun 1, 2017

Good works, looks solid. Will merge and test it in production.

@glaslos glaslos merged commit 27bf45d into mushorg:master Jun 1, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment