Eviction notification throws a null pointer exception.

[eboily]

Eviction notification throws a null pointer exception.

line 147 of org.infinispan.eviction.EvictionManagerImpl is

cacheNotifier.notifyCacheEntryEvicted(key, true, null);

the null third argument causes org.infinispan.notifications.cachelistener.CacheNotifier.notifyCacheEntryEvicted to launch a null exception at line 192.

The corrected line should be

cacheNotifier.notifyCacheEntryEvicted(key, true, getInvocationContext());

In the same way,

line 157 of org.infinispan.eviction.EvictionManagerImpl is

cacheNotifier.notifyCacheEntryEvicted(key, false, null);

and should be

cacheNotifier.notifyCacheEntryEvicted(key, false, getInvocationContext());

line 147 of org.infinispan.eviction.EvictionManagerImpl is

cacheNotifier.notifyCacheEntryEvicted(key, true, null);

the null third argument causes org.infinispan.notifications.cachelistener.CacheNotifier.notifyCacheEntryEvicted to launch a null exception at line 192.

I think the corrected line should be

cacheNotifier.notifyCacheEntryEvicted(key, true, getInvocationContext());

In the same way,

line 157 of org.infinispan.eviction.EvictionManagerImpl is

cacheNotifier.notifyCacheEntryEvicted(key, false, null);

and should be

cacheNotifier.notifyCacheEntryEvicted(key, false, getInvocationContext());

I did the patch on my own instance of infinispan and the solutions seems to work.

- Edouard

[vblagojevic]

What version of Infinispan are you referring to?

[eboily]

Hi Vladimir,

This part of the code is the same in 4.1.0-FINAL and 4.2.0-ALPHA2.

- Edouard

[vblagojevic]

Edouard,

Thanks for catching this one and reporting back. I've fixed trunk and 4.2 branch and added a unit test. In fact, all eviction unit tests are using cache eviction listener as of now. Reference JIRA is https://jira.jboss.org/browse/ISPN-674

Best regards,

Vladimir

[eboily]

Thanks Vladimir.

- Edouard