summaryrefslogtreecommitdiffstats
path: root/configserver
diff options
context:
space:
mode:
authorHarald Musum <musum@yahooinc.com>2021-11-30 16:34:15 +0100
committerHarald Musum <musum@yahooinc.com>2021-11-30 16:34:15 +0100
commit15e5edceeb2ef9207fd640ca2014c5c0bc273243 (patch)
treed9e9a06d1c812952673a9747f7a6091905c6e1b8 /configserver
parent900684d4f0ce62f0752860c4c7601ac042c7b1aa (diff)
Shutdown supervisor
Diffstat (limited to 'configserver')
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/ApplicationPackageMaintainer.java11
1 files changed, 8 insertions, 3 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/ApplicationPackageMaintainer.java b/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/ApplicationPackageMaintainer.java
index 0130721280f..c82f8093f63 100644
--- a/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/ApplicationPackageMaintainer.java
+++ b/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/ApplicationPackageMaintainer.java
@@ -43,6 +43,7 @@ public class ApplicationPackageMaintainer extends ConfigServerMaintainer {
private final ApplicationRepository applicationRepository;
private final File downloadDirectory;
private final ConfigserverConfig configserverConfig;
+ private final Supervisor supervisor = new Supervisor(new Transport("filedistribution-pool")).setDropEmptyBuffers(true);
private final FileDownloader fileDownloader;
ApplicationPackageMaintainer(ApplicationRepository applicationRepository,
@@ -54,7 +55,10 @@ public class ApplicationPackageMaintainer extends ConfigServerMaintainer {
this.configserverConfig = applicationRepository.configserverConfig();
this.downloadDirectory = new File(Defaults.getDefaults().underVespaHome(configserverConfig.fileReferencesDir()));
boolean useFileDistributionConnectionPool = Flags.USE_FILE_DISTRIBUTION_CONNECTION_POOL.bindTo(flagSource).value();
- this.fileDownloader = createFileDownloader(configserverConfig, useFileDistributionConnectionPool, downloadDirectory);
+ this.fileDownloader = createFileDownloader(configserverConfig,
+ useFileDistributionConnectionPool,
+ downloadDirectory,
+ supervisor);
}
@Override
@@ -95,8 +99,8 @@ public class ApplicationPackageMaintainer extends ConfigServerMaintainer {
private static FileDownloader createFileDownloader(ConfigserverConfig configserverConfig,
boolean useFileDistributionConnectionPool,
- File downloadDirectory) {
- Supervisor supervisor = new Supervisor(new Transport("filedistribution-pool")).setDropEmptyBuffers(true);
+ File downloadDirectory,
+ Supervisor supervisor) {
List<String> otherConfigServersInCluster = getOtherConfigServersInCluster(configserverConfig);
ConfigSourceSet configSourceSet = new ConfigSourceSet(otherConfigServersInCluster);
@@ -112,6 +116,7 @@ public class ApplicationPackageMaintainer extends ConfigServerMaintainer {
@Override
public void awaitShutdown() {
+ supervisor.transport().shutdown().join();
fileDownloader.close();
super.awaitShutdown();
}