Skip to content

[segment replication] Simplify the ckp verification for the primary shard relocation - #20609

Open
guojialiang92 wants to merge 4 commits into
opensearch-project:mainfrom
guojialiang92:dev/simplify-ckp-verification
Open

guojialiang92 wants to merge 4 commits into
opensearch-project:mainfrom
guojialiang92:dev/simplify-ckp-verification

add comment

6104bb8
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Apr 17, 2026 in 0s

73.37% (target 70.00%)

View this Pull Request on Codecov

73.37% (target 70.00%)

Details

Codecov Report

❌ Patch coverage is 78.94737% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 73.37%. Comparing base (e59c30c) to head (6104bb8).
⚠️ Report is 238 commits behind head on main.

Files with missing lines Patch % Lines
...org/opensearch/index/seqno/ReplicationTracker.java 0.00% 1 Missing ⚠️
...in/java/org/opensearch/index/shard/IndexShard.java 0.00% 1 Missing ⚠️
...s/replication/SegmentReplicationSourceService.java 88.88% 0 Missing and 1 partial ⚠️
.../indices/replication/SegmentReplicationTarget.java 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main   #20609      +/-   ##
============================================
- Coverage     73.37%   73.37%   -0.01%     
- Complexity    72197    72248      +51     
============================================
  Files          5798     5798              
  Lines        329816   329814       -2     
  Branches      47538    47536       -2     
============================================
- Hits         242001   241985      -16     
- Misses        68458    68488      +30     
+ Partials      19357    19341      -16     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.