[infinispan-issues] [JBoss JIRA] Resolved: (ISPN-718) Merge EvictionListener.preEvict() and postEvict() into one listener method.

[Trustin Lee (JIRA)]

     [ https://jira.jboss.org/browse/ISPN-718?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Trustin Lee resolved ISPN-718.
------------------------------

    Resolution: Done


Done: http://fisheye.jboss.org/changelog/Infinispan/?cs=2522
Diff somewhat unclear due to trivial code cleanup but it will get better as time goes by.

> Merge EvictionListener.preEvict() and postEvict() into one listener method.
> ---------------------------------------------------------------------------
>
>                 Key: ISPN-718
>                 URL: https://jira.jboss.org/browse/ISPN-718
>             Project: Infinispan
>          Issue Type: Task
>          Components: Core API
>    Affects Versions: 4.1.0.Final
>            Reporter: Trustin Lee
>            Assignee: Trustin Lee
>             Fix For: 4.2.0.BETA1
>
>
> EvictionListener.preEvict() is of dubious value (and actually the legacy of JBoss Cache) because a user cannot affect the behavior of eviction at all and a user can get notified on eviction via the postEvict() listener method.  I'd like to propose to change EvictionListener like this:
>     public interface EvictionListener<K, V> {
>         void onEntryEviction(K key, V value);
>     }

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: https://jira.jboss.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira