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

Remove CrdsValue::LeaderId #3188

Closed
mvines opened this issue Mar 8, 2019 · 0 comments
Closed

Remove CrdsValue::LeaderId #3188

mvines opened this issue Mar 8, 2019 · 0 comments

Comments

@mvines
Copy link
Contributor

mvines commented Mar 8, 2019

Problem

We're gossiping the leader id and that's now unnecessary

Proposed Solution

Remove CrdsValue::LeaderId and all associated gossip code.

@mvines mvines added this to the v0.13 Grandview milestone Mar 8, 2019
jeffwashington pushed a commit to jeffwashington/solana that referenced this issue Oct 29, 2024
* track zero lamport accounts

split off slot from shrink ancient stats

split slot stat

* populate zero lamport at start time

* pub

* comments

* comments

* pr fix

* fix

* test zero single ref init

* Update accounts-db/src/accounts_db.rs

Co-authored-by: Brooks <[email protected]>

* Update accounts-db/src/accounts_db.rs

Co-authored-by: Brooks <[email protected]>

* pr feedback reanem

* pr feedback outline fn

* pr: rename

* pr: rename

* pr: rename

* clippy

* fix stat

* no check for zero_lamport_single_ref_found

* num_slots_with_zero_lamport_accounts_added_to_shrink

* Update accounts-db/src/accounts_db.rs

Co-authored-by: Brooks <[email protected]>

* Update accounts-db/src/accounts_db.rs

Co-authored-by: Brooks <[email protected]>

* Update accounts-db/src/accounts_db.rs

Co-authored-by: Brooks <[email protected]>

* Update accounts-db/src/accounts_db.rs

Co-authored-by: Brooks <[email protected]>

* Update accounts-db/src/accounts_db.rs

Co-authored-by: Brooks <[email protected]>

* pr feedbacks

* rename

* Update accounts-db/src/accounts_db.rs

Co-authored-by: Brooks <[email protected]>

* pr

* pr

* pr

* Update accounts-db/src/accounts_db.rs

Co-authored-by: Brooks <[email protected]>

* fix build

* Update accounts-db/src/accounts_db.rs

Co-authored-by: Brooks <[email protected]>

* pr

---------

Co-authored-by: HaoranYi <[email protected]>
Co-authored-by: Brooks <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants