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

default to using packed ancient append vec #31657

Merged
merged 1 commit into from
May 16, 2023

Conversation

jeffwashington
Copy link
Contributor

@jeffwashington jeffwashington commented May 15, 2023

Problem

Switch to using packed ancient storages instead of appended ancient storages.
This has been thoroughly tested against mnb, kin, including with 300k slots INTO the current epoch against mnb.
We can remove the append code at some point soon.

Summary of Changes

This should have no effect except during testing until we enable the eliminate rewrites feature.

Fixes #

@codecov
Copy link

codecov bot commented May 15, 2023

Codecov Report

Merging #31657 (b2d3d95) into master (6172b0d) will increase coverage by 0.0%.
The diff coverage is 100.0%.

@@           Coverage Diff           @@
##           master   #31657   +/-   ##
=======================================
  Coverage    81.2%    81.3%           
=======================================
  Files         736      736           
  Lines      207455   207455           
=======================================
+ Hits       168658   168674   +16     
+ Misses      38797    38781   -16     

@jeffwashington jeffwashington marked this pull request as ready for review May 15, 2023 21:32
Copy link
Contributor

@brooksprumo brooksprumo left a comment

Choose a reason for hiding this comment

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

🥳

@jeffwashington jeffwashington merged commit db4b76d into solana-labs:master May 16, 2023
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