JdbcAccountPatchReviewStore: Remove unneeded else clauses
Change-Id: Idfcdd818861f18377bccf7b02ed59d8cbda2f519
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/schema/JdbcAccountPatchReviewStore.java b/gerrit-server/src/main/java/com/google/gerrit/server/schema/JdbcAccountPatchReviewStore.java
index 60360b4..70fbf5c 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/schema/JdbcAccountPatchReviewStore.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/schema/JdbcAccountPatchReviewStore.java
@@ -89,16 +89,18 @@
String url = cfg.getString(ACCOUNT_PATCH_REVIEW_DB, null, URL);
if (url == null || url.contains(H2_DB)) {
return new H2AccountPatchReviewStore(cfg, sitePaths);
- } else if (url.contains(POSTGRESQL)) {
- return new PostgresqlAccountPatchReviewStore(cfg, sitePaths);
- } else if (url.contains(MYSQL)) {
- return new MysqlAccountPatchReviewStore(cfg, sitePaths);
- } else if (url.contains(MARIADB)) {
- return new MariaDBAccountPatchReviewStore(cfg, sitePaths);
- } else {
- throw new IllegalArgumentException(
- "unsupported driver type for account patch reviews db: " + url);
}
+ if (url.contains(POSTGRESQL)) {
+ return new PostgresqlAccountPatchReviewStore(cfg, sitePaths);
+ }
+ if (url.contains(MYSQL)) {
+ return new MysqlAccountPatchReviewStore(cfg, sitePaths);
+ }
+ if (url.contains(MARIADB)) {
+ return new MariaDBAccountPatchReviewStore(cfg, sitePaths);
+ }
+ throw new IllegalArgumentException(
+ "unsupported driver type for account patch reviews db: " + url);
}
protected JdbcAccountPatchReviewStore(Config cfg, SitePaths sitePaths) {