]> git.ipfire.org Git - thirdparty/asterisk.git/commit
res_odbc: Implement a connection pool. 43/2943/3
authorJoshua Colp <jcolp@digium.com>
Thu, 2 Jun 2016 17:04:45 +0000 (14:04 -0300)
committerJoshua Colp <jcolp@digium.com>
Tue, 7 Jun 2016 14:58:03 +0000 (11:58 -0300)
commit321a9b128fdc8dd59ea720fd50691a3a32c0d934
tree1fca95e634da9bbf3224d498106b985daf53005b
parent93237209ebe05a849f8ae7a48d16b6f53676ea97
res_odbc: Implement a connection pool.

Testing has shown that our usage of UnixODBC is problematic
due to bugs within UnixODBC itself as well as the heavy weight
cost of connecting and disconnecting database connections, even
when pooling is enabled.

For users of UnixODBC 2.3.1 and earlier crashes would occur due
to insufficient protection of the disconnect operation. This was
fixed in UnixODBC 2.3.2 and above.

For users of UnixODBC 2.3.3 and higher a slow-down would occur
under heavy database use due to repeated connection establishment.
A regression is present where on each connection the database
configuration is cached again, with the cache growing out of
control.

The connection pool implementation present in this change helps
to mitigate these issues by reducing how much we connect and
disconnect database connections. We also solve the issue of
crashes under UnixODBC 2.3.1 by defaulting the maximum number of
connections to 1, returning us to the previous working behavior.
For users who may have a fixed version the maximum concurrent
connection limit can be increased helping with performance.

The connection pool works by keeping a list of active connections.
If the connection limit has not been reached a new connection is
established. If the connection limit has been reached then the
request waits until a connection becomes available before
continuing.

ASTERISK-26074 #close
ASTERISK-26054 #close

Change-Id: I6774bf4bac49a0b30242c76a09c403d2e856ecff
CHANGES
configs/samples/res_odbc.conf.sample
res/res_odbc.c