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

Performance metrics computation methodology #4041

Merged
merged 3 commits into from
Apr 27, 2019

Conversation

pgarg66
Copy link
Contributor

@pgarg66 pgarg66 commented Apr 27, 2019

Problem

No metrics methodology document

Summary of Changes

Added the document.

Fixes #3864

## Confimation Time

Each validator node maintains a list of active ledger forks that are visible
to the node. Whenever a fork's final tick is registered, the fork is frozen, and
Copy link
Member

Choose a reason for hiding this comment

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

That’s not correct. The validator confirmation time for a fork is based on the difference when a fork is created and when the first child fork that has a cumulative supermajority vote for the original fork s frozen.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I understand that fork is frozen when there's a cumulative supermajority vote. I don't understand why the child fork has to be frozen?

Copy link
Member

Choose a reason for hiding this comment

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

Frozen right now means that blocktree received all the blobs and the bank processed them. Validator confirmation time for a parent is based on when any descendant of the parent is frozen and that branch has a supermajority conforming the parent. Replay stage doesn’t compute a rolling vote total on incomplete forks.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ok. that makes sense. I had interpreted your last comment as one of the child fork has to receive supermajority vote. Just to confirm, this is when a fork is considered confirmed:

  1. The fork itself has received cumulative supermajority vote.
  2. One of the child of the fork has been frozen (i.e. received all it's blocks)

@pgarg66 pgarg66 merged commit 7f9a476 into solana-labs:master Apr 27, 2019
@pgarg66 pgarg66 deleted the dashboard branch April 27, 2019 23:37
solana-grimes pushed a commit that referenced this pull request Apr 27, 2019
lijunwangs pushed a commit to lijunwangs/solana that referenced this pull request Dec 13, 2024
* Define BankingTracer::create_channels()

* Make create_channels() take a bool
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.

No metrics methodology document
3 participants