summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHarald Musum <musum@verizonmedia.com>2020-09-11 19:00:30 +0200
committerHarald Musum <musum@verizonmedia.com>2020-09-11 19:00:30 +0200
commit305c24a5aa8771855ca0b3d51a50186df317f48c (patch)
tree4ae428b238fd861ef763262a5d6d558fa8f7e331
parent807761ce70f955f20fcc26dbd4d857e4f0c5fdb2 (diff)
Rename method
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionCache.java10
-rw-r--r--configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionRepository.java4
2 files changed, 4 insertions, 10 deletions
diff --git a/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionCache.java b/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionCache.java
index b7d78f11201..60fa037e99a 100644
--- a/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionCache.java
+++ b/configserver/src/main/java/com/yahoo/vespa/config/server/session/SessionCache.java
@@ -15,20 +15,14 @@ public class SessionCache<SESSIONTYPE extends Session> {
private final HashMap<Long, SESSIONTYPE> sessions = new HashMap<>();
- public synchronized void addSession(SESSIONTYPE session) {
- sessions.putIfAbsent(session.getSessionId(), session);
+ public synchronized void putSession(SESSIONTYPE session) {
+ sessions.put(session.getSessionId(), session);
}
synchronized void removeSession(long id) {
sessions.remove(id);
}
- /**
- * Gets a Session
- *
- * @param id session id
- * @return a session belonging to the id supplied, or null if no session with the id was found
- */
public synchronized SESSIONTYPE getSession(long id) {
return sessions.get(id);
}
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 cbfa59b26e4..3b2c03072c2 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
@@ -121,7 +121,7 @@ public class SessionRepository {
// ---------------- Local sessions ----------------------------------------------------------------
public synchronized void addLocalSession(LocalSession session) {
- localSessionCache.addSession(session);
+ localSessionCache.putSession(session);
long sessionId = session.getSessionId();
RemoteSession remoteSession = createRemoteSession(sessionId);
addSessionStateWatcher(sessionId, remoteSession, Optional.of(session));
@@ -263,7 +263,7 @@ public class SessionRepository {
}
public void addRemoteSession(RemoteSession session) {
- remoteSessionCache.addSession(session);
+ remoteSessionCache.putSession(session);
metrics.incAddedSessions();
}