Rasmus Johansson (Inactive) added a comment - 2012-12-07 14:39 Reaching out to user that experienced this to get more information.

Vladislav Vaintroub added a comment - 2012-12-09 17:32 I tried a lot of things (without DBCP, since I do not have any knowlegde about it), and could not see any difference in how setQueryTimeout/getQuerytimeout behave between ConnectorJ and MariaDB JDBC. I guess we need a runnable self-contained repro, be it DBCP or anything else.

Vladislav Vaintroub added a comment - 2012-12-16 01:46 I think I fixed it now. I did not try DBCP, but playing around with timeouts revealed JVM bug in socket timeouts that were used for query timeout. So this got reimplemented again with java.util.Timer.

George El Boustani added a comment - 2013-01-04 03:48 This fix solved the timeout exceptions, however it creates a resource leak as the Timer is not being cancelled and nulled. See attached patch

George El Boustani added a comment - 2013-01-04 03:51 Patch: === modified file 'src/main/java/org/mariadb/jdbc/MySQLStatement.java' --- src/main/java/org/mariadb/jdbc/MySQLStatement.java 2012-12-25 15:38:29 +0000 +++ src/main/java/org/mariadb/jdbc/MySQLStatement.java 2013-01-04 01:26:09 +0000 @@ -373,6 +373,9 @@ } } } + + cancelTimer(); + isClosed = true; } @@ -498,6 +501,8 @@ public void cancel() throws SQLException { try { protocol.cancelCurrentQuery(); + + cancelTimer(); } catch (QueryException e) { SQLExceptionMapper.throwException(e, connection, this); @@ -1182,4 +1187,11 @@ // TODO Auto-generated method stub return false; } + + private void cancelTimer() { + if (timer != null) { + timer.cancel(); + timer = null; + } + } }

George El Boustani added a comment - 2013-01-04 03:54 Before the patch over 3000 threads were running, and after it was down to 80

Vladislav Vaintroub added a comment - 2013-01-04 11:15 thanks. Would it be possible for you to paste a test case for "3000 threads are running"?

Vladislav Vaintroub added a comment - 2013-01-04 13:17 After reading docs for java.util.Timer - yes, you're right that there are excessive threads when using multiple timers. So ,the way to fix is it, is to use single (per-JVM) timer with multiple timerTasks , which I done in this patch : http://bazaar.launchpad.net/~maria-captains/mariadb-java-client/trunk/revision/376