Loading...

This issue belongs to an archived project. You can view it, but you can't modify it. Learn more

XML

Word

Printable

Steps to Reproduce:
Hide

SimpleCacheTest.java

public void testMerge() throws Exception { cache.put("existing", "foo"); Object mergeResult = cache.merge("existing", "bar", (oldValue, newValue) -> "" + oldValue + newValue); assertEquals("foobar", mergeResult); assertEquals("foobar", cache.get("existing")); mergeResult = cache.merge("existing", "bar", (oldValue, newValue) -> null); assertNull(mergeResult); assertNull(cache.get("existing")); mergeResult = cache.merge("notExisting", "foo", (oldValue, newValue) -> "" + oldValue + newValue); assertEquals("foo", mergeResult); assertEquals("foo", cache.get("notExisting")); }
Show
SimpleCacheTest.java public void testMerge() throws Exception { cache.put( "existing" , "foo" ); Object mergeResult = cache.merge( "existing" , "bar" , (oldValue, newValue) -> "" + oldValue + newValue); assertEquals( "foobar" , mergeResult); assertEquals( "foobar" , cache.get( "existing" )); mergeResult = cache.merge( "existing" , "bar" , (oldValue, newValue) -> null ); assertNull(mergeResult); assertNull(cache.get( "existing" )); mergeResult = cache.merge( "notExisting" , "foo" , (oldValue, newValue) -> "" + oldValue + newValue); assertEquals( "foo" , mergeResult); assertEquals( "foo" , cache.get( "notExisting" )); }
Estimated Difficulty:
Low
Git Pull Request:
https://github.com/infinispan/infinispan/pull/5017

Merge method is not correctly implemented in SimpleCache.
Merge should work this way :

if the key is not present, add the value
If the key is present, apply the function between the existing key's value and the given value and replace the key with the merged result
If the merge function returns null, remove the key

The case that should work as "putIfAbsent" does't work. Merge function is applied to null and the given value.

relates to

JDG-1143 Merge implementation is not correct on SimpleCache