[infinispan-issues] [JBoss JIRA] Commented: (ISPN-719) BoundedConcurrentHashMap.EvictionListener should have a bulk entry listener method.

[Manik Surtani (JIRA)]

    [ https://jira.jboss.org/browse/ISPN-719?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12558218#action_12558218 ] 

Manik Surtani commented on ISPN-719:
------------------------------------

Assigning to you, Trustin, since you volunteered on IRC.  :)

> BoundedConcurrentHashMap.EvictionListener should have a bulk entry listener method.
> -----------------------------------------------------------------------------------
>
>                 Key: ISPN-719
>                 URL: https://jira.jboss.org/browse/ISPN-719
>             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
>
>
> BoundedConcurrentHashMap.EvictionListener should have a bulk entry listener so that CacheStores can optimize passivation behavior.  Current BoundedConcurrentHashMap notifies EvictionListener for every evicted entry.  However, as you see from BoundedConcurrentHashMap.Segment.attemptEviction(..), eviction often happens in bulk. If these evicted entries are supposed to be passivated, it is often beneficial to begin a transaction because it often takes much less time to call commit() less often.  If N entries are evicted:
> without bulk eviction:
>     store.store(e1); -- each store implied a commit()
>     store.store(..);
>     store.store(eN);
> with bulk eviction:
>     store.begin(mods(e1, ..., eN), tx, true);

-- 
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