diff options
author | Edwin Kempin <ekempin@google.com> | 2015-12-03 08:19:41 +0000 |
---|---|---|
committer | Gerrit Code Review <noreply-gerritcodereview@google.com> | 2015-12-03 08:19:41 +0000 |
commit | c8c61f594fd7004c62ef14649913161c1d6d1da8 (patch) | |
tree | bd15f6451d49e39166acba3a88568367690833ab | |
parent | 455ed9cd27a16bf6991f04dcc57ef575dc4d5e75 (diff) | |
parent | 98e3448b1aaaf54e0bddfe9f1a539b3056fb4339 (diff) |
Merge changes from topic 'daemon-slave-mode-fixes' into stable-2.12
* changes:
ChangeCleanupConfig: Allow canonical web URL to be null
Daemon: Don't add change cleanup module when running as slave
Daemon: Don't add index commands when running as slave
-rw-r--r-- | gerrit-pgm/src/main/java/com/google/gerrit/pgm/Daemon.java | 10 | ||||
-rw-r--r-- | gerrit-server/src/main/java/com/google/gerrit/server/config/ChangeCleanupConfig.java | 7 |
2 files changed, 12 insertions, 5 deletions
diff --git a/gerrit-pgm/src/main/java/com/google/gerrit/pgm/Daemon.java b/gerrit-pgm/src/main/java/com/google/gerrit/pgm/Daemon.java index 39a5cbd833..ee1b11159b 100644 --- a/gerrit-pgm/src/main/java/com/google/gerrit/pgm/Daemon.java +++ b/gerrit-pgm/src/main/java/com/google/gerrit/pgm/Daemon.java @@ -278,7 +278,9 @@ public class Daemon extends SiteProgram { cfgInjector = createCfgInjector(); config = cfgInjector.getInstance( Key.get(Config.class, GerritServerConfig.class)); - initIndexType(); + if (!slave) { + initIndexType(); + } sysInjector = createSysInjector(); sysInjector.getInstance(PluginGuiceEnvironment.class) .setDbCfgInjector(dbInjector, cfgInjector); @@ -374,7 +376,9 @@ public class Daemon extends SiteProgram { } }); modules.add(new GarbageCollectionModule()); - modules.add(new ChangeCleanupRunner.Module()); + if (!slave) { + modules.add(new ChangeCleanupRunner.Module()); + } return cfgInjector.createChildInjector(modules); } @@ -415,7 +419,7 @@ public class Daemon extends SiteProgram { } modules.add(new DefaultCommandModule(slave, sysInjector.getInstance(DownloadConfig.class))); - if (indexType == IndexType.LUCENE) { + if (!slave && indexType == IndexType.LUCENE) { modules.add(new IndexCommandsModule()); } return sysInjector.createChildInjector(modules); diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/config/ChangeCleanupConfig.java b/gerrit-server/src/main/java/com/google/gerrit/server/config/ChangeCleanupConfig.java index 37a6869efd..b4b1865a43 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/config/ChangeCleanupConfig.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/config/ChangeCleanupConfig.java @@ -15,6 +15,7 @@ package com.google.gerrit.server.config; import com.google.common.base.Strings; +import com.google.gerrit.common.Nullable; import com.google.inject.Inject; import com.google.inject.Singleton; @@ -41,7 +42,7 @@ public class ChangeCleanupConfig { @Inject ChangeCleanupConfig(@GerritServerConfig Config cfg, - @CanonicalWebUrl String canonicalWebUrl) { + @CanonicalWebUrl @Nullable String canonicalWebUrl) { scheduleConfig = new ScheduleConfig(cfg, SECTION); abandonAfter = readAbandonAfter(cfg); abandonIfMergeable = @@ -61,7 +62,9 @@ public class ChangeCleanupConfig { if (Strings.isNullOrEmpty(abandonMessage)) { abandonMessage = DEFAULT_ABANDON_MESSAGE; } - abandonMessage = abandonMessage.replaceAll("\\$\\{URL\\}", webUrl); + if (!Strings.isNullOrEmpty(webUrl)) { + abandonMessage = abandonMessage.replaceAll("\\$\\{URL\\}", webUrl); + } return abandonMessage; } |