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

plug in new ledger, from #813 #830

Merged
merged 3 commits into from
Aug 3, 2018

Conversation

rob-solana
Copy link
Contributor

@rob-solana rob-solana commented Aug 2, 2018

accomplished:

  1. move write_stage to use LedgerWriter
  2. move fullnode to LedgerReader
    2a. change bin/fullnode to take a ledger directory
  3. move multinode tests to allow validators to have their own ledger directories
  4. add LedgerWriter to validators (in ReplicateStage)

next steps:

  1. add LedgerWindow to run_window_request
  2. delete dead EntryWriter code

@rob-solana rob-solana force-pushed the use-random-access-ledger branch 2 times, most recently from a052478 to fbf58ee Compare August 2, 2018 17:50
@rob-solana
Copy link
Contributor Author

@garious I want to know what you think of the placement of LedgerWriter in the validator's replicate stage

@rob-solana rob-solana force-pushed the use-random-access-ledger branch from fbf58ee to 3d60fb7 Compare August 2, 2018 18:32
@garious
Copy link
Contributor

garious commented Aug 2, 2018

Is the idea that EntryWriter will be deleted and so you're interacting with LedgerWriter directly from WriteStage?

@rob-solana
Copy link
Contributor Author

yeah, EntryWriter is not in use any more, will add removal to the TODOs... the big change is that validators are also keeping a persistent ledger, and it's being pooped out in ReplicateStage

@rob-solana rob-solana merged commit 384b486 into solana-labs:master Aug 3, 2018
@rob-solana rob-solana deleted the use-random-access-ledger branch August 3, 2018 04:50
vkomenda pushed a commit to vkomenda/solana that referenced this pull request Aug 29, 2021
jeffwashington pushed a commit to jeffwashington/solana that referenced this pull request Apr 17, 2024
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