FIFOPolicy

[kvistgaard]

I have implemented a simple FIFO eviction policy based on the LRU implementation and it seems to work OK. However the implementation could be simplified a great deal if the LRU classes were made more friendly for extension.

I've used the same configuration as the LRU implementation, but for FIFO an attribute named 'timeToLive' may be more appropriate than 'timeToIdleSeconds'. The advantage of having the same configuration for both policies is obvisously that the policies can easily be swapped.

I'd be hapy to supply a patch with the FIFOPolicy and the required changes to the LRU classes if there's any interest.


Regards and keep up with the excellent work!

Morten

[belaban]

Hi Morton,

why don't you contact Ben Wang and send him your changes, so we can integrate this with JBossCache.
Thanks for your contribution !

Bela

[ben.wang]

Morton,

Yes, please send it to [e-mail]. I realized the naming is not good and have been thinking whether I should rename it or not. Only thing is that it has been out for a while so I am not sure whether this will disrupt upgrade or not.

Thanks,

-Ben

[kvistgaard]

Hi Ben,

I'll send you a simple low impact patch (hopefully ;-) for the FIFO policy when I get back tonight. After that you can decide if you'd like anything else to be modified.

Later,

-Morten