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.

Sign up for GitHub

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

Jump to bottom

Optimize hash function #229

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from
Open

Optimize hash function #229

wants to merge 5 commits into from

Conversation

liudi4046
Copy link

@liudi4046 liudi4046 commented Jan 5, 2025 •
edited
Loading

  1. Make hash writer buffered to avoid multiple direct writing.
  2. Make extractTransactionsFromSlot asynchronously.
  3. Parallelize transaction hash computation in sequencingBatchStep
    In my macbook m3 pro, tps increased from 10.6 to 21.8

@liudi4046 liudi4046 changed the title Steven/optimize hash function Optimize hash function Jan 5, 2025
@liudi4046 liudi4046 requested review from giskook and zjg555543 January 6, 2025 01:51
@zjg555543
Copy link

Seams great
Could you share the comparison of pprof before and after optimization?

@zjg555543
Copy link

zjg555543 commented Jan 6, 2025 •
edited
Loading

Better to commit ckd-erigon upstream which help to reduce conflicts for future merges

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@giskook giskook Awaiting requested review from giskook

@zjg555543 zjg555543 Awaiting requested review from zjg555543

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@liudi4046 @zjg555543