summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHarald Musum <musum@verizonmedia.com>2020-09-17 21:50:02 +0200
committerHarald Musum <musum@verizonmedia.com>2020-09-17 21:50:02 +0200
commitabed9dbb485cf8a89ab03549bcf130ce977a936f (patch)
tree4cb4c76a1fe06c2473e1ad6e4c86f3448e049c15
parent6411525f438f153946af1cbabc0f1a33178c9bcf (diff)
Take lock before deleting
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionRepository.java4
1 files changed, 3 insertions, 1 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionRepository.java b/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionRepository.java
index 6e9e8ab59ff..4f577d8f62c 100644
--- a/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionRepository.java
+++ b/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionRepository.java
@@ -377,7 +377,9 @@ public class SessionRepository {
remoteSession.deactivate();
if (localSession == null) {
// This change will be picked up by directoryCache in this class, which will do the rest of the cleanup
- remoteSession.delete();
+ try (Lock lock = lock(remoteSession.getSessionId())) {
+ remoteSession.delete();
+ }
} else {
deleteLocalSession(localSession);
}