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

fix: memory leak from policy registration #68

Merged
merged 1 commit into from
Feb 8, 2024
Merged

fix: memory leak from policy registration #68

merged 1 commit into from
Feb 8, 2024

Conversation

YoshihitoAso
Copy link
Member

@YoshihitoAso YoshihitoAso self-assigned this Feb 8, 2024
@YoshihitoAso YoshihitoAso merged commit ec1feae into dev-2.3 Feb 8, 2024
5 checks passed
@YoshihitoAso YoshihitoAso deleted the fix/#56 branch February 8, 2024 11:55
@YoshihitoAso
Copy link
Member Author

YoshihitoAso commented Feb 9, 2024 •
edited

pprof

We profiled the memory usage of each client version on a local network (block size is approximately 19000).
We have succeeded in reducing the memory usage of validator.newDefaultSet.

v2.2.2
image

v2.3
image

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

@purplesmoke05 purplesmoke05 purplesmoke05 approved these changes

Assignees

@YoshihitoAso YoshihitoAso

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[BUG] Memory usage increases during long runs or block synchronization
2 participants
@YoshihitoAso @purplesmoke05