diff options
author | Harald Musum <musum@yahooinc.com> | 2021-11-30 16:34:15 +0100 |
---|---|---|
committer | Harald Musum <musum@yahooinc.com> | 2021-11-30 16:34:15 +0100 |
commit | 15e5edceeb2ef9207fd640ca2014c5c0bc273243 (patch) | |
tree | d9e9a06d1c812952673a9747f7a6091905c6e1b8 /configserver | |
parent | 900684d4f0ce62f0752860c4c7601ac042c7b1aa (diff) |
Shutdown supervisor
Diffstat (limited to 'configserver')
-rw-r--r-- | configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/ApplicationPackageMaintainer.java | 11 |
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(); } |