Evict callback about cachetools

[tkem]

Usually, popitem() is called to make room for new items when a cache is full, so I guess you could override that method to perform some action on the removed item(s).

[Iulian7]

Aha yea, I was just going through the code. Overriding popitem() seems to be the perfect place for this. Thanks.

[tkem]

You're welcome. May be hinting at this use case for popitem in the docs in the future.

[tkem]

After reviewing the docs, the fact that cache implementations do/should use popitem() to make room for new cache items is actually "hidden" in the Cache base class documentation. So perhaps putting this in the introduction section or at the top of the "Cache Implementations" section would make this more clear.
@Iulian7: If this was spelled out more prominently in the docs, maybe you wouldn't have had to go through the code? Any suggestions for making this more clear?

[Iulian7]

@tkem Yes if this was clearer in the docs I would not have gone through the code. Perhaps there should be an "extensibility" section in the docs?

[tkem]

I'll see what I can do.