commit fb92a5a50b6ce2abe94ca807937207210fd094cb
author Charles Chan <rascov@gmail.com> Thu Jul 05 16:46:11 2018 -0700
committer Charles Chan <rascov@gmail.com> Thu Jul 05 16:46:13 2018 -0700
tree 732d260feb05e25370e621a7259289379de39e28
parent 08815810a93e89ea5b40b498569d2c699e3bc981

Bugfix: DistributedPacketStore does not notify delegate in some cases

Assuming we execute two put operations with the same key but different value, i.e. put {key, value1} and put {key, value2}
Both put should return true

Change-Id: Iad8d68fa68e7b4ce37cdd3634d36144aa1b21afe

core/store/dist/src/main/java/org/onosproject/store/packet/impl/DistributedPacketStore.java

diff --git a/core/store/dist/src/main/java/org/onosproject/store/packet/impl/DistributedPacketStore.java b/core/store/dist/src/main/java/org/onosproject/store/packet/impl/DistributedPacketStore.java
index 1c02da9..1043c45 100644
--- a/core/store/dist/src/main/java/org/onosproject/store/packet/impl/DistributedPacketStore.java
+++ b/core/store/dist/src/main/java/org/onosproject/store/packet/impl/DistributedPacketStore.java
@@ -224,9 +224,7 @@
         }
 
         private boolean addInternal(PacketRequest request) {
-            Collection<? extends PacketRequest> values =
-                Versioned.valueOrNull(requests.putAndGet(key(request), request));
-            return values.size() == 1;
+            return requests.put(key(request), request);
         }
 
         private void remove(PacketRequest request) {