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: checkpoints getter and crash due to the usage of default MemTracker in MemoryDB #126

Merged
merged 9 commits into from
Sep 11, 2024
Merged

fix: checkpoints getter and crash due to the usage of default MemTracker in MemoryDB #126

merged 9 commits into from
Sep 11, 2024

Conversation

gshep
Copy link
Member

@gshep gshep commented Sep 10, 2024 •
edited
Loading

  • use NoopMemTracker
  • refactor init & sync_update_requires_replaying_back tests
  • add timeout parameter for Beacon-requests
  • add test case of getting checkpoint for a slot
  • fix Checkpoints::checkpoint method
  • relayer: log current batch iteration
  • fix sync_update::handle in case of replaying back is required

@gear-tech/dev

@gshep gshep self-assigned this Sep 10, 2024
@gshep gshep force-pushed the gshep/fix-relayer-crash branch from 086075a to 9b561ad Compare September 11, 2024 11:47
@gshep gshep changed the title Draft: refactor: use NoopMemTracker for MemoryDB fix: checkpoints getter and crash due to the usage of default MemTracker in MemoryDB Sep 11, 2024
@gshep gshep requested a review from mertwole September 11, 2024 12:01
@gshep gshep merged commit 26cc301 into main Sep 11, 2024
2 checks passed
@gshep gshep deleted the gshep/fix-relayer-crash branch September 11, 2024 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mertwole mertwole mertwole approved these changes

Assignees

@gshep gshep

Labels
bug refactoring
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@gshep @mertwole