Merge pull request #1553 from nlivens/mysql_driver_issue

Dynamically load drivers before creating our DB connectionsSolution to the 
mailing thread titled "MySQL : No suitable driver found for jdbc:mysql".
It doesn't harm that we explicitely load the MySQL driver, and for those which 
would use a commons-dbcp version < 1.4 this would fix it as well. Since JDBC 
4.0, the JDBC driver can auto-register itself, but for some weird cases (like 
ours), it's not working. Therefore we need to explicitly load the JDBC driver.

* pr/1553:
  Dynamic loading of DB driver + support for other DB providers

Signed-off-by: Will Stevens <williamstev...@gmail.com>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/b5606dcc
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/b5606dcc
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/b5606dcc

Branch: refs/heads/master
Commit: b5606dcce8b8834f956d24dad5d20bc8d5c001b9
Parents: de205c5 02c5d44
Author: Will Stevens <williamstev...@gmail.com>
Authored: Wed May 25 23:00:41 2016 -0400
Committer: Will Stevens <williamstev...@gmail.com>
Committed: Wed May 25 23:00:42 2016 -0400

----------------------------------------------------------------------
 build/replace.properties                        |  1 +
 client/tomcatconf/db.properties.in              |  3 ++
 .../snapshot/test/resources/db.properties       |  5 +++
 .../com/cloud/utils/db/TransactionLegacy.java   | 44 ++++++++++++++------
 framework/db/test/db.properties                 |  5 +++
 framework/jobs/test/resources/db.properties     |  5 +++
 packaging/centos63/replace.properties           |  1 +
 packaging/centos7/replace.properties            |  1 +
 packaging/centos7/tomcat7/db.properties         |  5 +++
 packaging/debian/replace.properties             |  1 +
 packaging/fedora20/replace.properties           |  1 +
 packaging/fedora21/replace.properties           |  1 +
 .../globodns/test/resources/db.properties       |  5 +++
 .../test/resources/db.properties                |  5 +++
 server/test/resources/db.properties             |  5 +++
 .../iam/plugin/test/resources/db.properties     |  5 +++
 .../iam/server/test/resources/db.properties     |  5 +++
 tools/devcloud-kvm/pom.xml                      |  2 +-
 tools/devcloud/pom.xml                          |  2 +-
 tools/devcloud4/pom.xml                         |  2 +-
 usage/conf/db.properties.in                     |  1 +
 usage/test/resources/db.properties              |  5 +++
 utils/conf/db.properties                        |  5 +++
 23 files changed, 100 insertions(+), 15 deletions(-)
----------------------------------------------------------------------


Reply via email to