[jbosscache-issues] [JBoss JIRA] Commented: (JBCACHE-1582) ExpirationAlgorithm doesn't evict nodes that have been created in a long Tx

[Galder Zamarreno (JIRA)]

    [ https://jira.jboss.org/browse/JBCACHE-1582?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12546070#action_12546070 ] 

Galder Zamarreno commented on JBCACHE-1582:
-------------------------------------------

I'm delighted to see the fix working fine for you. Fixing this was far from easy. Had to do a few iterations to get it right :)

I can't say right now when you'll be able to get a patched JBC. I'm gonna be committing the fix right now and will get in touch with Dennis to figure out what would be best, if a one-off patch, or a new JBC release.



> ExpirationAlgorithm doesn't evict nodes that have been created in a long Tx 
> ----------------------------------------------------------------------------
>
>                 Key: JBCACHE-1582
>                 URL: https://jira.jboss.org/browse/JBCACHE-1582
>             Project: JBoss Cache
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: Eviction
>    Affects Versions: 3.2.5.GA
>            Reporter: Nicolas Filotto
>            Assignee: Manik Surtani
>         Attachments: Expiration_and_LRU_solution_based_on_command_and_transaction_transfer.patch, TestEvictionAlgorithms.java, TestExpirationAlgorithm.java, TestExpirationAlgorithm.java, TestExpirationAlgorithm.java
>
>
> I noticed that I had a memory leak in my application when I use the ExpirationAlgorithm (maybe this bug happens with other algorithms) due to some nodes that were never evicted. After a deeper investigation, I noticed that no EvictionEntry was created for those nodes since the method getExpiration returns null (which also means that cache.peek(fqn, false) returns null). I finally found the problem, that is because if the Expiration Thread wakes up before that the related Tx is committed, it doesn't find the node into the cache so it considers it as removed but in fact it is just because the Tx has not yet been committed. This has for consequence that the node will never be evicted and thus create a memory leak.
> Please find the related unit test as attached file.

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