[IGNITE-3621] Make GridCacheTtlManager singleton

• Type: Bug
• Status: Resolved
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 1.6
• Fix Version/s: 2.8
• Component/s: cache
• Labels: performance
• Assignee: Unassigned
• Reporter: Eduard Shangareev
• Created: 03/Aug/16 10:41
• Updated: 10/Oct/19 13:11
• Resolved: 27/Sep/16 11:24

Description

Now every cache has own TTL manager, which creates CleanupWorker = new extra thread. This can cause to extra hundreds of threads (redundant context switches = performance penalty).

Also, under IGNITE-3513 every put can enter critical section to notify worker. Obviously, it is not good from performance point of view.

So, my proposal is next:
1. Expiration should be done on every cache action (on exit thread which updates cache should invoke expire).
2. TtlManager will exist only in one instance.
3. CleanupWorker will be the only backup if there is no cache activity. It will wake up with some period to check for work (500 ms, for example).

Moreover, now we keep on-heap pending entries even if a cache is kept off-head. At least, this issue needs discussion.

Issue Links

• is related to: IGNITE-3513 Cleanup worker is placed in the Thread's waiting queue using Thread.sleep method
• links to: GitHub Pull Request #1066
• links to: GitHub Pull Request #1066