summaryrefslogtreecommitdiffstats
path: root/configserver
diff options
context:
space:
mode:
authorHarald Musum <musum@verizonmedia.com>2020-05-06 20:14:37 +0200
committerGitHub <noreply@github.com>2020-05-06 20:14:37 +0200
commit29b263529e3baaaaf69044ff58a8d260bd604bdd (patch)
treeb8fe52c611c057b6b4f2db989b71304e8bf62aae /configserver
parent1fdbeaba57862dd81e91d3794a5fdbf48a359075 (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.java7
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);
}
}