Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix: fix some panic error when use replication#865

Merged
hezhangjian merged 1 commit intomainfrom
fix_raftlog
Apr 20, 2025
Merged

fix: fix some panic error when use replication#865
hezhangjian merged 1 commit intomainfrom
fix_raftlog

Conversation

@xiangyu5632
Copy link
Member

When you use openGemini replication mode, a panic may occur after a period of normal system running. The reason is that when reading the raft log file, the read-write lock used causes the file offset to be incorrect, which ultimately prevents the WAL data from being cleaned up normally. The WAL file may become larger and larger, but the data will still normally be synchronized between replicas.

@codecov
Copy link

codecov bot commented Apr 20, 2025

Codecov Report

Attention: Patch coverage is 81.03448% with 11 lines in your changes missing coverage. Please review.

Project coverage is 76.79%. Comparing base (d5c540b) to head (b77dbba).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #865      +/-   ##
==========================================
+ Coverage   76.72%   76.79%   +0.06%     
==========================================
  Files         668      668              
  Lines      141536   141569      +33     
==========================================
+ Hits       108597   108711     +114     
+ Misses      26574    26508      -66     
+ Partials     6365     6350      -15     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hezhangjian hezhangjian merged commit c965047 into main Apr 20, 2025
11 checks passed
@hezhangjian hezhangjian deleted the fix_raftlog branch April 20, 2025 06:22
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