From e7d40f930f8e85d67b3d235ca26c7f2f076d6a58 Mon Sep 17 00:00:00 2001 From: Rossen Stoyanchev Date: Mon, 3 Feb 2020 20:59:30 +0000 Subject: [PATCH] Remove synchronized block As per the Javadoc of ConcurrentHashMap its computeIfAbsent implementation is atomic and hence already synchronized internally, so we can remove the surrounding synchronization block. See gh-24470 --- .../annotation/RequestMappingHandlerAdapter.java | 11 +++-------- 1 file changed, 3 insertions(+), 8 deletions(-) diff --git a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/RequestMappingHandlerAdapter.java b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/RequestMappingHandlerAdapter.java index ead2f88c8a..66049d310e 100644 --- a/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/RequestMappingHandlerAdapter.java +++ b/spring-webmvc/src/main/java/org/springframework/web/servlet/mvc/method/annotation/RequestMappingHandlerAdapter.java @@ -821,14 +821,9 @@ public class RequestMappingHandlerAdapter extends AbstractHandlerMethodAdapter * (never {@code null}). */ private SessionAttributesHandler getSessionAttributesHandler(HandlerMethod handlerMethod) { - Class handlerType = handlerMethod.getBeanType(); - SessionAttributesHandler sessionAttrHandler = this.sessionAttributesHandlerCache.get(handlerType); - if (sessionAttrHandler == null) { - synchronized (this.sessionAttributesHandlerCache) { - sessionAttrHandler = this.sessionAttributesHandlerCache.computeIfAbsent(handlerType, type -> new SessionAttributesHandler(type, this.sessionAttributeStore)); - } - } - return sessionAttrHandler; + return this.sessionAttributesHandlerCache.computeIfAbsent( + handlerMethod.getBeanType(), + type -> new SessionAttributesHandler(type, this.sessionAttributeStore)); } /**