Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat/utils: lru cache #3420

Merged
merged 2 commits into from
Aug 4, 2023
Merged

feat/utils: lru cache #3420

merged 2 commits into from
Aug 4, 2023

Conversation

kanishkatn
Copy link
Contributor

@kanishkatn kanishkatn commented Aug 2, 2023

Changes

Implements LRUCache.

It is being used in disputes-subsystem and few other places in the polkadot code. This is the crate polkadot is using https://docs.rs/lru/latest/lru/.

Tests

go test ./...

Primary Reviewer

@timwu20

@kanishkatn kanishkatn force-pushed the feat/k/lrucache branch 2 times, most recently from c69a5a2 to 797bf46 Compare August 2, 2023 20:01
@codecov
Copy link

codecov bot commented Aug 2, 2023

Codecov Report

Merging #3420 (4a9be45) into development (2eb6313) will increase coverage by 0.02%.
The diff coverage is 90.90%.

❗ Current head 4a9be45 differs from pull request most recent head c0324f6. Consider uploading reports for the commit c0324f6 to get more accurate results

Additional details and impacted files
@@               Coverage Diff               @@
##           development    #3420      +/-   ##
===============================================
+ Coverage        51.02%   51.04%   +0.02%     
===============================================
  Files              231      232       +1     
  Lines            30753    30786      +33     
===============================================
+ Hits             15691    15716      +25     
- Misses           13383    13389       +6     
- Partials          1679     1681       +2     

Copy link
Contributor

@jimjbrettj jimjbrettj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@kanishkatn kanishkatn merged commit e0fa0d9 into development Aug 4, 2023
@kanishkatn kanishkatn deleted the feat/k/lrucache branch August 4, 2023 05:51
Copy link

🎉 This PR is included in version 0.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants