-
Actually this was still open (no version set…).
Please add unit-tests to all methods just to make sure it works correctly.
Could you also add some javadoc, especially to
LRUCacheWithFuture
explaining motivation and benefits of this implementation (as opposed to plainLRUCache
for example)?
Type |
Task
|
Priority |
Normal
|
Assignee | |
Version |
tigase-server-8.4.0
|
Spent time |
0
|
Iterations
-
tigase-server-8.4.0 Closed
Issue Votes (0)
Currently, our caching implementation is not LRU which would be best for us (we have last recently loaded in cache), so it may happen that frequently used node configuration, subscribers and affiliations list will get removed from the memory. Additionally, our caching mechanism may cause loading of the same node configuration if many threads will ask about it at the same time (loading will not cause other threads to wait, instead it will force them to load the data).