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

[TieredStorage] Rename account-index to index-block #33928

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

yhchiang-sol
Copy link
Contributor

Problem

The current tiered-storage code uses "account-index" to call index-block.
This could lead to confusion especially as we start giving each offset/position/index a specific type.

Summary of Changes

This PR renames all structs/variables that use account-index to refer to index-block.

@codecov
Copy link

codecov bot commented Oct 30, 2023

Codecov Report

Merging #33928 (5126087) into master (808f67a) will increase coverage by 0.0%.
Report is 1 commits behind head on master.
The diff coverage is 100.0%.

❗ Current head 5126087 differs from pull request most recent head 90ae7ec. Consider uploading reports for the commit 90ae7ec to get more accurate results

@@           Coverage Diff            @@
##           master   #33928    +/-   ##
========================================
  Coverage    81.9%    81.9%            
========================================
  Files         809      809            
  Lines      218257   218060   -197     
========================================
- Hits       178784   178653   -131     
+ Misses      39473    39407    -66     

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.

Love it, thanks!

@yhchiang-sol yhchiang-sol merged commit 6624a09 into solana-labs:master Nov 6, 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.

2 participants