plone / guillotina

Python AsyncIO data API to manage billions of resources
https://guillotina.readthedocs.io/en/latest/
Other
187 stars 51 forks source link

ported: prevent caching objects bigger than max_cache_record_size #1088

Closed lferran closed 3 years ago

lferran commented 3 years ago

original pr to 5.x branch: https://github.com/plone/guillotina/pull/1085

codecov-io commented 3 years ago

Codecov Report

Merging #1088 (8295ea9) into master (570cb6c) will increase coverage by 0.1%. The diff coverage is 100.0%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #1088     +/-   ##
========================================
+ Coverage    94.5%   94.5%   +0.1%     
========================================
  Files         341     341             
  Lines       29241   29273     +32     
========================================
+ Hits        27625   27657     +32     
  Misses       1616    1616             
Impacted Files Coverage Δ
guillotina/contrib/cache/strategy.py 95.0% <100.0%> (+0.1%) :arrow_up:
guillotina/tests/cache/test_cache_memory.py 100.0% <100.0%> (ø)
bloodbare commented 3 years ago

@lferran are you going to port memcache contrib package ?

lferran commented 3 years ago

@bloodbare yes, but I'll do it in a separate PR if you don't mind.