[server][router] Removing unneeded Optional to reduce memory allocati…

…on (#796)
linkedin · Dec 12, 2023 · b54fbb9 · b54fbb9
1 parent a91cabe
commit b54fbb9
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/...y4/alpini-netty4-base/src/main/java/com/linkedin/alpini/netty4/misc/BasicHttpRequest.java b/...y4/alpini-netty4-base/src/main/java/com/linkedin/alpini/netty4/misc/BasicHttpRequest.java
@@ -20,7 +20,6 @@
 import java.io.IOException;
 import java.io.ObjectOutputStream;
 import java.util.Objects;
-import java.util.Optional;
 import java.util.UUID;
 import java.util.concurrent.atomic.AtomicInteger;
 
@@ -231,7 +230,10 @@ static AttributeMap getAttibuteMap(HttpMessage httpMessage) {
   }
 
   AttributeMap attributeMap() {
-    return Optional.ofNullable(_attributes).orElseGet(() -> _attributes = new DefaultAttributeMap());
+    if (_attributes == null) {
+      _attributes = new DefaultAttributeMap();
+    }
+    return _attributes;
   }
 
   @SuppressWarnings("unchecked")