[GitHub] trafficserver pull request: TS-4187: Keep the currently_open conne...

2016-05-03 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/trafficserver/pull/583 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

[GitHub] trafficserver pull request: TS-4187: Keep the currently_open conne...

2016-05-03 Thread shinrich
Github user shinrich commented on the pull request: https://github.com/apache/trafficserver/pull/583#issuecomment-216679006 I think you could do a TSQA test. Configure with global pools. Run a set of requests through. Wait a while for things to drain out. Verify that the current_

[GitHub] trafficserver pull request: TS-4187: Keep the currently_open conne...

2016-04-24 Thread jpeach
Github user jpeach commented on the pull request: https://github.com/apache/trafficserver/pull/583#issuecomment-214042536 This looks good to me. I think it should be reasonable to construct a simple integration test for this? --- If your project is set up for it, you can reply to thi

[GitHub] trafficserver pull request: TS-4187: Keep the currently_open conne...

2016-04-19 Thread shinrich
GitHub user shinrich opened a pull request: https://github.com/apache/trafficserver/pull/583 TS-4187: Keep the currently_open connections in sync with open sockets This is an improvement on the original fix for TS-4187. The original fix would in some case over count the open socket