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

Reduce the default maxChainBytesPerSyncAttempt to 50MB #303

Open
Wayonb opened this issue Oct 20, 2021 · 1 comment
Open

Reduce the default maxChainBytesPerSyncAttempt to 50MB #303

Wayonb opened this issue Oct 20, 2021 · 1 comment

Comments

@Wayonb
Copy link
Contributor

Wayonb commented Oct 20, 2021

Describe the bug
During the sync of testnet, the catapult-client will timeout when pulling blocks. This is because certain block are have a lot of transactions and on slower networks takes a while to pull 100MB.

Should lower the maxChainBytesPerSyncAttempt to 50MB

To Reproduce
To reproduce the behavior:

  1. Try to sync testnet
@dusanjp
Copy link

dusanjp commented Oct 23, 2021

With the settings created by the current bootstrap, it is difficult to exceed 130681 blocks with testnet (except for nodes with abundant cpu resources).
After this 130681 block, it is the time when nft-drive started test operation.
Furthermore, we are conducting a stress test at the end of August, but it is also a heavy block here, which is hindering synchronization. It is about 330000 blocks or later.

Here, I tried with the following settings based on the advice of those who noticed it:
maxChainBytesPerSyncAttempt: 10MB
messageSynchronizationMaxResponseSize: 5MB
blockDisruptorMaxMemorySize: 1000MB

If node is stopped for a long time and then restarted or a new node is started without making the above settings, synchronization could not be completed (except for nodes with abundant cpu resources).

The default values for these settings are:
maxChainBytesPerSyncAttempt: 100MB
messageSynchronizationMaxResponseSize: 20MB
blockDisruptorMaxMemorySize: 300MB

Since nft-drive will be running on mainnet in the near future
In the next bootstrap, I think it is necessary to adjust the default values of these values.

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

2 participants