Replaced computeIfAbsent() with simpler alg: costly lambda creation

belaban · Aug 27, 2024 · 3733c34 · 3733c34
1 parent 4e7b30d
commit 3733c34
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/src/org/jgroups/util/MaxOneThreadPerSender.java b/src/org/jgroups/util/MaxOneThreadPerSender.java
@@ -84,7 +84,13 @@ public MessageTable() {
         }
 
         protected Entry get(final Address sender, boolean multicast) {
-            return map.computeIfAbsent(sender, s -> new Entry(sender, multicast, tp.getClusterNameAscii()));
+            Entry e=map.get(sender);
+            if(e != null)
+                return e;
+            // not so elegant, but avoids lambda allocation!
+            Entry tmp=map.putIfAbsent(sender, (e=new Entry(sender, multicast, tp.getClusterNameAscii())));
+            return tmp!= null? tmp: e;
+            // return map.computeIfAbsent(sender, s -> new Entry(sender, multicast, tp.getClusterNameAscii()));
         }
 
         protected void clear() {map.clear();}