Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Free up the term 'replicate' for exclusive use in ledger replication #2057

Merged
merged 3 commits into from
Dec 10, 2018

Conversation

garious
Copy link
Contributor

@garious garious commented Dec 7, 2018

Problem

Replication code is using name "storage" to avoid interfering with the term replicate that's been awkwardly used by the TVU to describe the stage that replays the ledger.

Summary of Changes

  • Free up the term 'replicate' for exclusive use in ledger replication
  • Rename ReplicateStage to ReplayStage
  • Align Sockets field names with ContactInfo. (transaction -> tpu, replicate -> tvu)

cc: #1409

@garious
Copy link
Contributor Author

garious commented Dec 7, 2018

@pgarg66, added you to review in the context of counter names. There's no tests protecting those strings. The test suite lets me change them with no regard for the implications.

@pgarg66
Copy link
Contributor

pgarg66 commented Dec 7, 2018

@garious , changing the string names would also require to update metrics dashboard.
Also, we use common dashboard between master and older branches. So it would mean separate dashboard for legacy branches (beta/stable).

@garious garious force-pushed the rename-replicate branch 2 times, most recently from 359671e to 54a1e5e Compare December 8, 2018 03:04
@garious garious removed the request for review from pgarg66 December 9, 2018 02:39
@garious
Copy link
Contributor Author

garious commented Dec 10, 2018

@sakridge, @sagar-solana, still waiting on you guys. Are you okay with these name changes?

Copy link
Member

@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

Copy link
Contributor

@sagar-solana sagar-solana left a comment

Choose a reason for hiding this comment

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

lgtm

@garious garious merged commit 9556a9b into solana-labs:master Dec 10, 2018
vkomenda pushed a commit to vkomenda/solana that referenced this pull request Aug 29, 2021
* Fix signer parameters (staker, manager, depositor, fee-payer, and
  token-owner)
* On deposit / withdraw / add / remove validator, show the validator
  stake account and where the stake is going to
brooksprumo pushed a commit to brooksprumo/solana that referenced this pull request Jul 9, 2024
Bumps solana_rbpf to v0.8.2 and removes switch_to_new_elf_parser.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants