Remove keys from cache on exception

[laky55555]

What do these changes do?

Remove key from cache as soon as the task fails.
Currently, it removes the key from the cache on subsequent retrival.

Benefits:

• less unnecessary data in cache leading to smaller footprint and if the cache is full, less evictions keeping it more performant
• remove memory leak on exception happening

Are there changes in behavior for the user?

No (only noticable change should be less data in cache stats cache_info())

Related issue number

This PR can replace 595 one as there will be no more exeptions in cache stored.
This one introduces a bit more changes so if it is going in the wrong direction other one can be merged instead to remove memory leak only.

Checklist

• I think the code is well written
• Unit tests for the changes exist
• Documentation reflects the changes

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.93%. Comparing base (f60bf71) to head (811a582).
Report is 60 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #595      +/-   ##
==========================================
+ Coverage   96.88%   96.93%   +0.05%     
==========================================
  Files          12       12              
  Lines         770      783      +13     
  Branches       91       94       +3     
==========================================
+ Hits          746      759      +13     
  Misses         22       22              
  Partials        2        2              

Flag	Coverage Δ
unit	96.80% <100.00%> (+0.05%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.