Please note: BoneCP is considered deprecated at this point (it's pretty old
now!). You are strongly encouraged to switch to HikariCP which offers
better performance and is more stable.

On Fri, Dec 19, 2014 at 1:42 PM, ggerritsen <email address hidden>
wrote:
>
> Hi Selvakumar,
>
> Is the max_connections variable set to 85 in your mysql config?
>
> mysql> show variables where Variable_name = "max_connections";
>
>
> In any case, your situation shows that the amount of connections that
> bonecp holds is different than the amount of connections in mysql created
> by the application, which is wrong. The patch could fix this, but it's also
> possible you have a different problem.
>
> Instead of applying the patch you could also consider updating to a
> later version of bonecp, such as bonecp-parent-0.8.0-beta1 or any newer
> version (see https://github.com/wwadge/bonecp/releases).
>
> Hope this helps!
>
> Kind regards,
>
> Geert
>
> --
> You received this bug notification because you are subscribed to BoneCP.
> https://bugs.launchpad.net/bugs/999114
>
> Title:
>   Connection is not closed when releasing if connectionHandle is expired
>
> Status in BoneCP - Java Database Connection Pool:
>   Fix Released
>
> Bug description:
>   Hello,
>
>   Last week, we observed some unexpected behavior using bonecp. Our max
> connections were set to 10 (5 max per partition, 2 partitions), but over
> the course of half an hour, the number of connections reached 40.
>   However, bonecp reported that it only had 10 connections. When garbage
> collection was performed, the 'extra' connections were cleaned up.
>
>   This seemed strange, so we started looking into it. Eventually, we
>   found the culprit. In BoneCP.internalReleaseConnection(), the
>   following happens:
>
>   {code}
>   protected void internalReleaseConnection(ConnectionHandle
> connectionHandle) throws SQLException {
>     // snip
>
>     if (connectionHandle.isExpired() || (!this.poolShuttingDown &&
> connectionHandle.isPossiblyBroken()
>       && !isConnectionHandleAlive(connectionHandle))){
>
>      ConnectionPartition connectionPartition =
> connectionHandle.getOriginatingPartition();
>      postDestroyConnection(connectionHandle);
>
>      maybeSignalForMoreConnections(connectionPartition);
>      connectionHandle.clearStatementCaches(true);
>      return; // don't place back in queue - connection is broken or
> expired.
>     }
>
>     // snip
>   }
>   {code}
>
>   As you can see, this code does not actually _close_ the connection, it
> assumes it is already closed and therefore calls the
> postDestroyConnection() method.
>   However, this code might also be executed if the connection is not
> closed yet. This can happen if the connectionHandle is expired (i.e., the
> maxConnectionAge has passed), in which case the connection might still be
> open and will not be closed by here. Which leads such a connection to be
> released from the pool (i.e. BoneCP does not see it anymore), but still be
> open (which explains the increase in connections past the maxConnection
> level).
>
>   If you agree that this is a bug, we can supply a patch for it.
>
>   Regards, Geert
>
>   P.S. We use bonecp 0.7.1.RELEASE
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/bonecp/+bug/999114/+subscriptions
>