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

Conversation

@jeffwashington
Copy link
Contributor

@jeffwashington jeffwashington commented Jun 23, 2021

Problem

Validator startup time is painful. Untar on a snapshot is expensive.

Summary of Changes

Untar in parallel. This is part of a series of changes revisiting the pipeline from snapshot through account index generation.
Fixes #

@jeffwashington
Copy link
Contributor Author

metrics

@mvines
Copy link
Contributor

mvines commented Jun 23, 2021

Oh very cool!

@jeffwashington jeffwashington force-pushed the excess40 branch 4 times, most recently from c56956e to cb886b6 Compare June 24, 2021 14:04
@jeffwashington jeffwashington marked this pull request as ready for review June 24, 2021 14:18
@jeffwashington jeffwashington requested a review from sakridge June 24, 2021 14:18
Copy link
Contributor

@sakridge sakridge left a comment

Choose a reason for hiding this comment

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

lgtm

@codecov
Copy link

codecov bot commented Jun 24, 2021

Codecov Report

Merging #18184 (d9c00ca) into master (6b62ba0) will decrease coverage by 0.0%.
The diff coverage is 34.8%.

@@            Coverage Diff            @@
##           master   #18184     +/-   ##
=========================================
- Coverage    82.3%    82.3%   -0.1%     
=========================================
  Files         433      433             
  Lines      120917   120940     +23     
=========================================
+ Hits        99609    99619     +10     
- Misses      21308    21321     +13     

@jeffwashington jeffwashington merged commit 31ec986 into solana-labs:master Jun 24, 2021
mergify bot pushed a commit that referenced this pull request Jun 24, 2021
* untar in parallel

* make enum for 'ignore' return value

(cherry picked from commit 31ec986)
jeffwashington added a commit that referenced this pull request Jun 25, 2021
* untar in parallel

* make enum for 'ignore' return value

(cherry picked from commit 31ec986)

Co-authored-by: Jeff Washington (jwash) <[email protected]>
jeffwashington added a commit to jeffwashington/solana that referenced this pull request Jun 30, 2021
* untar in parallel

* make enum for 'ignore' return value
@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.

3 participants