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

cherrypick memory leak fixes to v0.22 testnet #1519

Merged
merged 3 commits into from
Oct 22, 2021
Merged

cherrypick memory leak fixes to v0.22 testnet #1519

merged 3 commits into from
Oct 22, 2021

Conversation

zhangchiqing
Copy link
Member

@zhangchiqing zhangchiqing commented Oct 22, 2021 •
edited

zhangchiqing and others added 2 commits October 22, 2021 11:31
@zhangchiqing
fix potential mem-leak from hotstuff (#1514)
cde9b0b
@Kay-Zee @zhangchiqing @synzhu
Kan/clean up secondary result index (#1515)
7fcf45c 
* Remove from secondary index

* Add tests for caches

* add concurrency test

* Apply suggestions from code review

Co-authored-by: Simon Zhu <simon.zsiyan@gmail.com>

Co-authored-by: Leo Zhang (zhangchiqing) <zhangchiqing@gmail.com>
Co-authored-by: Simon Zhu <simon.zsiyan@gmail.com>
@zhangchiqing zhangchiqing merged commit 21aeee5 into v0.22-testnet Oct 22, 2021
@zhangchiqing zhangchiqing deleted the leo/v0.22-testnet-memory-leak-fix branch October 22, 2021 23:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Kay-Zee Kay-Zee Kay-Zee approved these changes

@AlexHentschel AlexHentschel Awaiting requested review from AlexHentschel AlexHentschel is a code owner

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

Successfully merging this pull request may close these issues.

None yet
2 participants
@zhangchiqing @Kay-Zee