Skip to content

Conversation

@wjblanke
Copy link
Contributor

@wjblanke wjblanke commented Dec 10, 2025

DO NOT MERGE plz


Note

Removes implicit DB close from Mempool.__del__ and explicitly closes the mempool SQLite connection in MempoolManager.shut_down() and after slow-path new_peak reinit, with added logging.

  • Mempool lifecycle:
    • Remove implicit DB close by deleting Mempool.__del__.
    • Explicitly close mempool DB connection:
      • On shutdown in MempoolManager.shut_down() (with info log).
      • After slow-path mempool rebuild in MempoolManager.new_peak() (old_pool._db_conn.close()).

Written by Cursor Bugbot for commit 2f2b200. This will update automatically on new commits. Configure here.

@wjblanke wjblanke requested a review from a team as a code owner December 10, 2025 00:34
@wjblanke wjblanke added the Fixed Required label for PR that categorizes merge commit message as "Fixed" for changelog label Dec 10, 2025
@github-actions github-actions bot added the merge_conflict Branch has conflicts that prevent merge to main label Dec 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Fixed Required label for PR that categorizes merge commit message as "Fixed" for changelog merge_conflict Branch has conflicts that prevent merge to main

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant