Skip to content
This repository has been archived by the owner on Jan 22, 2025. It is now read-only.

Increment usage count for loaded programs and call eviction #30900

Merged
merged 1 commit into from
Mar 27, 2023

Conversation

pgarg66
Copy link
Contributor

@pgarg66 pgarg66 commented Mar 27, 2023

Problem

The new LoadedPrograms cache eviction logic is not being exercised.

Summary of Changes

  • Increment usage count of a LoadedProgram when it is executed.
  • Call cache eviction code after a batch of transaction has been executed.

Fixes #

@codecov
Copy link

codecov bot commented Mar 27, 2023

Codecov Report

Merging #30900 (7a52f88) into master (3bc670a) will decrease coverage by 0.1%.
The diff coverage is 100.0%.

@@            Coverage Diff            @@
##           master   #30900     +/-   ##
=========================================
- Coverage    81.5%    81.5%   -0.1%     
=========================================
  Files         726      726             
  Lines      204907   204911      +4     
=========================================
- Hits       167064   167031     -33     
- Misses      37843    37880     +37     

@pgarg66 pgarg66 requested a review from Lichtso March 27, 2023 12:58
@pgarg66 pgarg66 merged commit 0df28f6 into solana-labs:master Mar 27, 2023
@pgarg66 pgarg66 deleted the loaded-programs-hits branch March 27, 2023 13:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

2 participants