Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Ensure the caches always return Deferreds #5583

Closed
wants to merge 14 commits into from

Conversation

hawkowl
Copy link
Contributor

@hawkowl hawkowl commented Jul 1, 2019

This stores Deferreds consistently inside the cache.

@codecov
Copy link

codecov bot commented Jul 1, 2019

Codecov Report

Merging #5583 into develop will decrease coverage by 0.5%.
The diff coverage is 86.36%.

@@             Coverage Diff             @@
##           develop    #5583      +/-   ##
===========================================
- Coverage    63.15%   62.64%   -0.51%     
===========================================
  Files          328      328              
  Lines        35901    35880      -21     
  Branches      5914     5907       -7     
===========================================
- Hits         22672    22477     -195     
- Misses       11604    11752     +148     
- Partials      1625     1651      +26

@codecov
Copy link

codecov bot commented Jul 1, 2019

Codecov Report

Merging #5583 into develop will increase coverage by 0.02%.
The diff coverage is 66.66%.

@@             Coverage Diff             @@
##           develop    #5583      +/-   ##
===========================================
+ Coverage    63.15%   63.17%   +0.02%     
===========================================
  Files          328      328              
  Lines        35901    35896       -5     
  Branches      5914     5915       +1     
===========================================
+ Hits         22672    22679       +7     
+ Misses       11604    11590      -14     
- Partials      1625     1627       +2

@richvdh
Copy link
Member

richvdh commented Jul 1, 2019

I think this is fixing #5582, among other things

@erikjohnston
Copy link
Member

This does break the length stuff in lrucache, which I'll have to come back to (if it's actually useful?)

I believe we heavily rely on this feature to ensure that our cache memory usage is proportional to the cache size? Without it we'll end up with unpredictable memory usage.

@hawkowl hawkowl self-assigned this Jul 2, 2019
@richvdh
Copy link
Member

richvdh commented Mar 19, 2020

I fear this has bit-rotten beyond repair :/

@richvdh richvdh closed this Mar 19, 2020
@richvdh richvdh deleted the hawkowl/cache-return-deferred branch April 6, 2022 12:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants