Details
-
Sub-task
-
Status: Resolved
-
Minor
-
Resolution: Duplicate
-
None
-
None
-
None
Description
findbugs is complaining
if (sessionsById.get(id) == null) { SessionImpl s = new SessionImpl(id, sessionTimeout); sessionsById.put(id, s); }
is not atomic for the gap between get() and put().
I suggest using putIfAbsent() instead.
Attachments
Attachments
Issue Links
- duplicates
-
ZOOKEEPER-1982 Refactor (touch|add)Session in SessionTrackerImpl.java
- Resolved