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

refactor reconstruct_accountsdb_from_fields #17987

Merged
merged 1 commit into from
Jun 21, 2021

Conversation

jeffwashington
Copy link
Contributor

@jeffwashington jeffwashington commented Jun 15, 2021

Problem

startup is slow. Working on pipelining stages involved.

Summary of Changes

Refactor reconstructing a single storage file after extraction.
Fixes #

@jeffwashington jeffwashington force-pushed the excess7 branch 2 times, most recently from f62cb07 to aadceca Compare June 16, 2021 01:53
@codecov
Copy link

codecov bot commented Jun 16, 2021

Codecov Report

Merging #17987 (30d4de7) into master (0a81c37) will increase coverage by 0.0%.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##           master   #17987   +/-   ##
=======================================
  Coverage    82.3%    82.3%           
=======================================
  Files         434      434           
  Lines      121192   121195    +3     
=======================================
+ Hits        99767    99790   +23     
+ Misses      21425    21405   -20     

@jeffwashington jeffwashington marked this pull request as ready for review June 17, 2021 19:07
@jeffwashington jeffwashington requested a review from jbiseda June 17, 2021 19:07
jbiseda
jbiseda previously approved these changes Jun 17, 2021
@mergify mergify bot dismissed jbiseda’s stale review June 17, 2021 19:51

Pull request has been modified.

@jeffwashington jeffwashington merged commit ae82e4e into solana-labs:master Jun 21, 2021
mergify bot pushed a commit that referenced this pull request Jun 21, 2021
jeffwashington added a commit that referenced this pull request Jun 21, 2021
(cherry picked from commit ae82e4e)

Co-authored-by: Jeff Washington (jwash) <[email protected]>
@brooksprumo brooksprumo mentioned this pull request Aug 23, 2021
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