Hikari + MSSQL 7.2.1 having issues

Hello All,

We have an spring boot application that is using HikariCP and mssql 6.x libraries and everything seems working fine.

We started moving to the latest version of mssql, which is 7.2.1 and started observing issues connecting to the database, once in a while under load test.

On the initial research, we found that the SharedTimer thingy that got introduced is killing us.

2019-04-17_00:27:40.315 [42122f1e7f861f55:42122f1e7f861f55:0] [Root=1-5cb672fb-8a9936f01f1e83407ebf0024] [qtp1673605040-61] ERROR com.zaxxer.hikari.pool.HikariPool - HikariPool-278 - Exception during pool initialization. java.util.concurrent.RejectedExecutionException: Task java.util.concurrent.ScheduledThreadPoolExecutor$ScheduledFutureTask@746dc536 rejected from java.util.concurrent.ScheduledThreadPoolExecutor@7dd4eba[Terminated, pool size = 0, active threads = 0, queued tasks = 0, completed tasks = 0] at java.util.concurrent.ThreadPoolExecutor$AbortPolicy.rejectedExecution(ThreadPoolExecutor.java:2063) at java.util.concurrent.ThreadPoolExecutor.reject(ThreadPoolExecutor.java:830) at java.util.concurrent.ScheduledThreadPoolExecutor.delayedExecute(ScheduledThreadPoolExecutor.java:326) at java.util.concurrent.ScheduledThreadPoolExecutor.schedule(ScheduledThreadPoolExecutor.java:533) at com.microsoft.sqlserver.jdbc.SharedTimer.schedule(SharedTimer.java:102)

Any help on how to fix is highly appreciated.

[xhumanoid]

It is because you have not thread-safe implementation of SharedTimer introduced in #920

public static synchronized SharedTimer getTimer() use lock on class instance public synchronized void removeRef() use lock on timer object instance

so possible situation when

1. first thread start removeRef, shutdown executer and execute executor = null;
2. second thread receive the same object over getTimer and execute schedule, but timer almost finished/shutdowned

refCount also updated in lines 63 and 84 without any memory barriers or synchronization, so nobody can guaranty what the final result will be =)

[cheenamalhotra]

Hi @TheHydGuy @xhumanoid

I’ve created PR #1046 to address thread safety issues as discussed here, please take a look and let me know if I missed something.