amphp / sql-common / 32

39%

push

travis-ci-com

Connection pool pop() or push() should modify to release idle connection. (#3)

In source code, the pop() method invoke idle->shift() to get top of pool list connection to use, and push return it back to the bottom of list.

In timeoutWatcher first check bottom of the list connection, if not idle will do nothing.

If connection pool has expanded at peak, when the traffic drops back to a low peak, as long as there is one query in each idle time, all connection will never released!!! but only one or two connection is enough now.

So I think idle->push() should change to idle->unshift(), then real idle connection in bottom will be released in time, or timeoutWatcher check top, and pop() change to real idle->pop().

See also https://github.com/amphp/mysql/issues/99

1 of 1 new or added line in 1 file covered. (100.0%)

113 of 288 relevant lines covered (39.24%)

3.4 hits per line