diff options
author | Harald Musum <musum@verizonmedia.com> | 2020-05-06 20:14:37 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-05-06 20:14:37 +0200 |
commit | 29b263529e3baaaaf69044ff58a8d260bd604bdd (patch) | |
tree | b8fe52c611c057b6b4f2db989b71304e8bf62aae /configserver | |
parent | 1fdbeaba57862dd81e91d3794a5fdbf48a359075 (diff) |
Revert "Revert "Reduce the time we keep expired remotes sessions and fix comment""
Diffstat (limited to 'configserver')
-rw-r--r-- | configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/SessionsMaintainer.java | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/SessionsMaintainer.java b/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/SessionsMaintainer.java index c4f0c5fdd5d..c0211fb6e91 100644 --- a/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/SessionsMaintainer.java +++ b/configserver/src/main/java/com/yahoo/vespa/config/server/maintenance/SessionsMaintainer.java @@ -27,11 +27,10 @@ public class SessionsMaintainer extends ConfigServerMaintainer { protected void maintain() { applicationRepository.deleteExpiredLocalSessions(); - // Expired remote sessions are not expected to exist, they should have been deleted when - // a deployment happened or when the application was deleted. We still see them from time to time, - // probably due to some race or another bug + // Expired remote sessions are sessions that belong to an application that have external deployments that + // are no longer active if (hostedVespa) { - Duration expiryTime = Duration.ofDays(30); + Duration expiryTime = Duration.ofDays(7); applicationRepository.deleteExpiredRemoteSessions(expiryTime); } } |