Skip to content
This repository was archived by the owner on Aug 8, 2023. It is now read-only.

Add p.Remove(conn)#3

Merged
ibigbug merged 1 commit intomasterfrom
feat/remove-conn
Oct 12, 2016
Merged

Add p.Remove(conn)#3
ibigbug merged 1 commit intomasterfrom
feat/remove-conn

Conversation

@ibigbug
Copy link
Copy Markdown
Owner

@ibigbug ibigbug commented Oct 12, 2016

Remove conn immediately since it's know closed by remote and can not get back to pool.

@coveralls
Copy link
Copy Markdown

coveralls commented Oct 12, 2016

Coverage Status

Coverage increased (+0.08%) to 96.117% when pulling ec32594 on feat/remove-conn into 424890b on master.

2 similar comments
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.08%) to 96.117% when pulling ec32594 on feat/remove-conn into 424890b on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.08%) to 96.117% when pulling ec32594 on feat/remove-conn into 424890b on master.

@coveralls
Copy link
Copy Markdown

coveralls commented Oct 12, 2016

Coverage Status

Coverage increased (+0.08%) to 96.117% when pulling d759e45 on feat/remove-conn into 424890b on master.

@ibigbug ibigbug merged commit 330af9a into master Oct 12, 2016
@ibigbug ibigbug deleted the feat/remove-conn branch October 12, 2016 10:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants