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

BEP-151: Decommission Decentralized Exchange on BNB Beacon Chain #151

Merged
merged 8 commits into from Jul 11, 2022

Conversation

owen-reorg
Copy link
Contributor

@owen-reorg owen-reorg commented Jun 16, 2022

BEP-151: Decommission Decentralized Exchange on BNB Beacon Chain

1. Summary

This BEP proposes to securely and smoothly decommission the build-in decentralized exchange on BNB Beacon Chain.

2. Abstract

BNB Beacon Chain’s primary focus, its native decentralized application ("dApp") BNB DEX, has demonstrated its low-latency matching with large capacity headroom. With the development of BNB Smart Chain and AMM-based decentralized exchanges running well on it, BNB DEX has less usage and liquidity. After implementing this BEP, the DEX module will be disabled, which will give Beacon Chain more computing power for the future computing and governance focuses.

3. Status

Draft

4. Motivation

BNB Beacon Chain(BC) is a blockchain developed by the BNB Chain community that implements a vision of a decentralized exchange (DEX) for digital assets. The heart of Beacon Chain is a highly performant matching engine built on distributed consensus that aims to replicate the < 1 second trading efficiency of current centralized exchanges.

Since BNB Smart Chain's(BSC) launch to mainnet in 2020, the AMM based decentralized exchanges gain great success on BSC which even steal the thunder of order-book based DEX on BC. BC and BSC is a dual chain structure, with the evolution of this structure, BC plays more like a Beacon Chain that helps to enhance the security of BSC as a staking and governance layer. It is not suitable for a Beacon Chain to sustain a high-performance DEX any longer, especially when there is another DEX with massive adoption in the same ecosystem.

Decommissioning the DEX module will reduce the need for system resources significantly for both validators and full nodes. The demand for liquidation can be fulfilled by BNB Smart Chain or the upcoming zkBAS.

5. Specification

After the implementation of this BEP, transactions related to DEX(including NewOrderMsg, ListMsg, ListMiniMsg) will return an error to disable listing new trading pairs or placing any new orders.
All the existing orders will be canceled either by traders or by 3 days timeout(or 30 days for orders in the best 500 price levels, according to BEP-67). Thus there will be no more DEX transactions on Beacon Chain.

5.1 Safety

After the implementation of this BEP, there will be no more new orders.
The existing orders which are not matched yet will not be matched anymore.
They can be canceled by the traders directly or by timeout automatically after a period of time, and refunded to the traders.
All the funds are SAFU.

5.2 Impact

Transactions MsgSubmitProposal with type of ProposalTypeListTradingPair or ProposalTypeDelistTradingPair are used for submitting proposals to list new trading pairs or delist trading pairs.

They might be not useful any longer but still valid after the BEP. It's not encouraged to send these transactions anymore.

6. License

The content is licensed under CC0.

@owen-reorg owen-reorg changed the title [WIP] Beacon Chain Dex Prune [WIP] BEP-151: Beacon Chain Dex Prune Jun 16, 2022
@que1414
Copy link

que1414 commented Jun 16, 2022 via email

@owen-reorg owen-reorg changed the title [WIP] BEP-151: Beacon Chain Dex Prune BEP-151: Beacon Chain Dex Prune Jun 17, 2022
@owen-reorg owen-reorg changed the title BEP-151: Beacon Chain Dex Prune BEP-151: Beacon Chain DEX Prune Jun 17, 2022
@owen-reorg owen-reorg changed the title BEP-151: Beacon Chain DEX Prune BEP-151: Decommission Decentralized Exchange on BNB Beacon Chain Jun 20, 2022
@owen-reorg owen-reorg dismissed a stale review via 1d25f3d July 11, 2022 05:44
unclezoro
unclezoro previously approved these changes Jul 11, 2022
@unclezoro unclezoro merged commit dd55e6e into bnb-chain:master Jul 11, 2022
@Bladez1753
Copy link

owen-reorg:bc-dex-prune

@Arielek9
Copy link

Arielek9 commented Sep 7, 2022

Wtf

@blaaauw
Copy link

blaaauw commented Sep 11, 2022 via email

Copy link

@Debu976116 Debu976116 left a comment

Choose a reason for hiding this comment

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

Bnb

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.

None yet

7 participants