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

create remove_if_slot_list_empty to define accounts index behavior #19581

Merged
merged 1 commit into from
Sep 2, 2021

Conversation

jeffwashington
Copy link
Contributor

@jeffwashington jeffwashington commented Sep 2, 2021

Problem

This function has complicated interactions with an on-disk index. This refactoring makes the second step more clear.

Summary of Changes

Fixes #

@jeffwashington jeffwashington marked this pull request as ready for review September 2, 2021 21:09
@codecov
Copy link

codecov bot commented Sep 2, 2021

Codecov Report

Merging #19581 (8cc6f58) into master (f5388cf) will decrease coverage by 0.0%.
The diff coverage is 88.8%.

@@            Coverage Diff            @@
##           master   #19581     +/-   ##
=========================================
- Coverage    82.5%    82.4%   -0.1%     
=========================================
  Files         468      468             
  Lines      131542   131547      +5     
=========================================
- Hits       108575   108521     -54     
- Misses      22967    23026     +59     

@jeffwashington jeffwashington merged commit 2874f6b into solana-labs:master Sep 2, 2021
frits-metalogix added a commit to identity-com/solana that referenced this pull request Nov 24, 2021
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.

2 participants