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

change prefetch thread num #830

Merged
merged 1 commit into from
Apr 1, 2022
Merged

Conversation

flywukong
Copy link
Contributor

@flywukong flywukong commented Apr 1, 2022

Description

since the main thread has become faster after using sharedStorage , we need more prefetch thread num to make the prefetch is faster than main process.

Rationale

After after adding some metric on prefetch therad , the metric shows that the prefetcher with two threads has 5-20% faster than main process while prefetcher with three threads has become 20-40% faster than main process .

The yellow line is using 3 thread
image

Performance

The two test machine is on AWS , the two nodes has worked in fullsync mode and has caught up to the latest block

The cost of process a block show that the processing cost of main process with three prefetch threads has become 3%-10% faster than two prefetch threads

image

Compare to the processing cost of main process prefetcher with 3 thread and 4 thread , the cost of process a block is nearly same , so 3 thread is enough for prefetcher
image

Changes

Notable changes:

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

@unclezoro unclezoro merged commit c57b02c into bnb-chain:develop Apr 1, 2022
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