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

Node configuration #42

Open
szynwelski opened this issue Jul 7, 2023 · 1 comment
Open

Node configuration #42

szynwelski opened this issue Jul 7, 2023 · 1 comment
Assignees
Milestone

Comments

@szynwelski
Copy link
Contributor

szynwelski commented Jul 7, 2023

The purpose of the task is to go through all the configuration options and set the appropriate values.

Useful links:

@szynwelski
Copy link
Contributor Author

szynwelski commented Jul 10, 2023

We are introducing the following configuration changes:

Peer Exchange Reactor

There is no need to discover and connect with new peers until the network is open. Initially, the list of nodes will be static and specified in persistent_peers list. More about PEX can be read here.

Empty Blocks

We don't want to produce empty blocks (see #37). In case of no transactions, it is sufficient to generate a block occasionally (e.g., once every half a minute). Unfortunately, changing the values of the create_empty_blocks and create_empty_blocks_interval parameters is not enough to achieve this. Empty blocks will still be produced if the app_hash changes with every block. Currently, the app_hash changes each time due to modules that modify something in storage through BeginBlocker or EndBlocker (e.g., distribution or slashing). As part of task #36, we will need to consider whether we want to disable/modify these modules. On the other hand, there is a high chance that transactions will be sent to the sequencer frequently, so perhaps it is not worth focusing on this functionality.

Inflation

At the moment, the sequencer network will operate on a temporary token, and we do not anticipate any transfers between accounts. Therefore, inflation is not necessary and has been disabled.

The above changes were introduced in the PR.

@szynwelski szynwelski self-assigned this Jul 10, 2023
@szynwelski szynwelski added this to the First phase milestone Jul 10, 2023
szynwelski added a commit that referenced this issue Jul 28, 2023
…ration

Change node configuration: pex, inflation, empty blocks #42
@szynwelski szynwelski modified the milestones: First phase, Second phase Nov 22, 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

No branches or pull requests

1 participant