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.

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

feat(merkle): Automate debug info collection and fix entities checkpo… #14078

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

VolodymyrBg
Copy link
Contributor

…int tracking

This PR addresses two TODOs in the merkle stage implementation:

  1. Adds automated debug information collection for state root errors

    • Introduces collect_state_root_debug_info helper function
    • Automatically gathers database stats with checksum
    • Formats debug report with all necessary information for issue reporting
    • Makes it easier for users to submit comprehensive bug reports
  2. Implements proper entities checkpoint tracking during unwind

    • Updates the entities checkpoint to reflect the actual state after unwinding
    • Properly tracks processed and total entries
    • Ensures accurate progress reporting during the unwind process

The changes improve the developer and user experience by:

  • Simplifying the process of reporting state root issues
  • Providing more accurate progress tracking during unwind operations
  • Making the codebase more maintainable by removing TODOs

Testing:

  • Added unit tests for the new debug info collection function
  • Verified checkpoint tracking in existing unwind tests
  • Manual testing of the debug report format

Closes #XXXX (reference to tracking issue if exists)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant