Skip to content

Commit

Permalink
set the driver classname explicitly
Browse files Browse the repository at this point in the history
  • Loading branch information
syjer committed Nov 7, 2016
1 parent 4d5efa3 commit 6307d17
Show file tree
Hide file tree
Showing 2 changed files with 35 additions and 0 deletions.
1 change: 1 addition & 0 deletions src/main/java/alfio/config/DataSourceConfiguration.java
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,7 @@ public DataSource getDataSource(Environment env, PlatformProvider platform) thro
dataSource.setJdbcUrl(platform.getUrl(env));
dataSource.setUsername(platform.getUsername(env));
dataSource.setPassword(platform.getPassword(env));
dataSource.setDriverClassName(platform.getDriveClassName(env));
int maxActive = platform.getMaxActive(env);

dataSource.setMaximumPoolSize(maxActive);
Expand Down
34 changes: 34 additions & 0 deletions src/main/java/alfio/config/support/PlatformProvider.java
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,11 @@ public String getDialect(Environment env) {
return PGSQL;
}

@Override
public String getDriveClassName(Environment env) {
return POSTGRESQL_DRIVER;
}

@Override
public boolean isHosting(Environment env) {
return ofNullable(env.getProperty("ELEPHANTSQL_URI")).isPresent() || ofNullable(env.getProperty("VCAP_SERVICES")).filter(s -> s.contains("elephantsql")).isPresent();
Expand Down Expand Up @@ -109,6 +114,11 @@ public String getDialect(Environment env) {
return PGSQL;
}

@Override
public String getDriveClassName(Environment env) {
return POSTGRESQL_DRIVER;
}

@Override
public boolean isHosting(Environment env) {
return ofNullable(env.getProperty("OPENSHIFT_APP_NAME")).isPresent();
Expand Down Expand Up @@ -141,6 +151,11 @@ public String getDialect(Environment env) {
return isMySql(env) ? MYSQL : PGSQL;
}

@Override
public String getDriveClassName(Environment env) {
return isMySql(env) ? MYSQL_DRIVER : POSTGRESQL_DRIVER;
}

@Override
public boolean isHosting(Environment env) {
//check if json object for services is returned
Expand Down Expand Up @@ -185,6 +200,11 @@ public String getDialect(Environment env) {
return PGSQL;
}

@Override
public String getDriveClassName(Environment env) {
return POSTGRESQL_DRIVER;
}

@Override
public boolean isHosting(Environment env) {
return ofNullable(env.getProperty("DYNO")).isPresent();
Expand Down Expand Up @@ -222,6 +242,12 @@ public String getDialect(Environment env) {
return PGSQL;
}


@Override
public String getDriveClassName(Environment env) {
return POSTGRESQL_DRIVER;
}

@Override
public boolean isHosting(Environment env) {
return ofNullable(env.getProperty("DB_ENV_POSTGRES_DB")).isPresent();
Expand Down Expand Up @@ -263,6 +289,11 @@ private boolean isMySql(Environment env) {
return ofNullable(env.getProperty("RDS_PORT")).filter("3306"::equals).isPresent();
}

@Override
public String getDriveClassName(Environment env) {
return isMySql(env) ? MYSQL_DRIVER : POSTGRESQL_DRIVER;
}

};

private static final String POSTGRESQL_DRIVER = "org.postgresql.Driver";
Expand All @@ -284,6 +315,9 @@ public String getPassword(Environment env) {
return env.getRequiredProperty("datasource.password");
}

public String getDriveClassName(Environment env) {
return env.getRequiredProperty("datasource.driver");
}

public String getDialect(Environment env) {
return env.getRequiredProperty("datasource.dialect");
Expand Down

0 comments on commit 6307d17

Please sign in to comment.