Skip to content

fix: remove naturally finality#1651

Merged
NathanBSC merged 1 commit intobnb-chain:developfrom
node-real:remove_naturally_finality
May 30, 2023
Merged

fix: remove naturally finality#1651
NathanBSC merged 1 commit intobnb-chain:developfrom
node-real:remove_naturally_finality

Conversation

@NathanBSC
Copy link
Copy Markdown
Contributor

@NathanBSC NathanBSC commented May 25, 2023

Description

remove naturally finality

Rationale

naturally finality is alias of probabilistic finality in the code.

  1. The probabilistic finality is only probabilistic. In extreme cases, it may be reverted without penalty, so it is not suitable to acknowledge it through the interface
  2. Different application scenarios of probabilistic finality may choose different numbers of blocks. For example, 15 is recommended, while the main station uses 21 for deposits and withdrawals. but only a fixed number can be specified in the code
  3. The Ethereum client will also degenerate to a probabilistic finality when the vote fails, but the number of blocks is not specified in the code; Bitcoin has probabilistic finality, and the number of blocks is not specified in the code

Example

add an example CLI or API response...

Changes

Notable changes:

  • add each change in a bullet point here
  • ...

@NathanBSC NathanBSC force-pushed the remove_naturally_finality branch from dc987dc to 7ec23fa Compare May 25, 2023 06:45
@NathanBSC NathanBSC force-pushed the remove_naturally_finality branch from 7ec23fa to 02a9dd6 Compare May 25, 2023 07:52
@emailtovamos
Copy link
Copy Markdown
Contributor

What exactly is the reason for removal of NaturallyFinalizedDist ? I didn't quite understand the rationale given.

@NathanBSC NathanBSC requested a review from brilliant-lx May 26, 2023 01:06
@NathanBSC
Copy link
Copy Markdown
Contributor Author

What exactly is the reason for removal of NaturallyFinalizedDist ? I didn't quite understand the rationale given.

reason updated

@NathanBSC NathanBSC requested a review from unclezoro May 26, 2023 01:19
@NathanBSC NathanBSC merged commit 3d8753c into bnb-chain:develop May 30, 2023
@NathanBSC NathanBSC deleted the remove_naturally_finality branch May 30, 2023 01:56
setunapo added a commit to setunapo/bsc-erigon that referenced this pull request May 31, 2023
setunapo added a commit to node-real/bsc-erigon that referenced this pull request May 31, 2023
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