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

changes from main #11492

Merged
merged 1 commit into from
Aug 6, 2024
Merged

changes from main #11492

merged 1 commit into from
Aug 6, 2024

Conversation

dvovk
Copy link
Member

@dvovk dvovk commented Aug 5, 2024

Changes:

  • Added functionality to display download metrics such as size, total time, and average rate.
  • Improved command for diagnostic stages and avoided printing empty tables.
  • Implemented a diagnostics database for saving state data.
  • Replaced the speed test library with a custom version using Erigon's infrastructure.
  • Limited speed test runs and stopped tests upon snapshot download completion.
  • Fixed issues with downloaded percentages exceeding 100% and JSON unmarshalling errors. Improved error messages for better debugging.
  • Continued listening for events even after initial completion, considering the two-part download process.
  • Refactored stages to include substages and enhanced logging for header chain downloads.
  • Bug Fixes
  • Fixed race conditions in test cases and improved diagnostics test client.
  • Optimized database writes, reduced unnecessary warnings, and adjusted log levels.
  • Refactored mutex usage for network peers and other critical sections.

@dvovk dvovk added this to the 2.60.6-fixes milestone Aug 5, 2024
@dvovk dvovk requested review from awskii and AskAlexSharov and removed request for awskii August 5, 2024 15:29
@AskAlexSharov AskAlexSharov merged commit 08447f4 into release/2.60 Aug 6, 2024
6 checks passed
@AskAlexSharov AskAlexSharov deleted the diagupde3 branch August 6, 2024 03:02
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.

2 participants