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

Simplifies adding uncleaned pubkeys during index generation #34007

Conversation

brooksprumo
Copy link
Contributor

Problem

Index generation identifies uncleaned pubkeys. The current impl has redundant code. The Entry API can make this simpler.

Summary of Changes

Simplify 😸

@brooksprumo brooksprumo self-assigned this Nov 9, 2023
@brooksprumo brooksprumo marked this pull request as ready for review November 9, 2023 19:16
@brooksprumo brooksprumo requested a review from HaoranYi November 9, 2023 19:16
Copy link

codecov bot commented Nov 9, 2023

Codecov Report

Merging #34007 (06026b4) into master (bba6ea2) will decrease coverage by 0.1%.
Report is 4 commits behind head on master.
The diff coverage is 100.0%.

@@            Coverage Diff            @@
##           master   #34007     +/-   ##
=========================================
- Coverage    81.9%    81.9%   -0.1%     
=========================================
  Files         811      811             
  Lines      219347   219344      -3     
=========================================
- Hits       179732   179719     -13     
- Misses      39615    39625     +10     

Copy link
Contributor

@HaoranYi HaoranYi left a comment

Choose a reason for hiding this comment

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

lgtm

@brooksprumo brooksprumo merged commit 9f25f67 into solana-labs:master Nov 9, 2023
@brooksprumo brooksprumo deleted the generate-index/uncleaned-pubkeys-entry branch November 9, 2023 21:06
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.

2 participants