Skip to content

VDiff: add --no-samples option to vdiff show - #20870

Merged
mattlord merged 12 commits into
vitessio:mainfrom
pedroalb:pedroalb/vdiff-show-summary-only
Sep 1, 2026
Merged

VDiff: add --no-samples option to vdiff show#20870
mattlord merged 12 commits into
vitessio:mainfrom
pedroalb:pedroalb/vdiff-show-summary-only

test: remove explanatory narrative comments from vdiff no-samples tests

fc1d8fa
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Aug 31, 2026 in 0s

70.90% (+1.23%) compared to 70c7a72

View this Pull Request on Codecov

70.90% (+1.23%) compared to 70c7a72

Details

Codecov Report

❌ Patch coverage is 84.61538% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 70.90%. Comparing base (70c7a72) to head (fc1d8fa).
⚠️ Report is 516 commits behind head on main.

Files with missing lines Patch % Lines
...d/vtctldclient/command/vreplication/vdiff/vdiff.go 33.33% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##             main   #20870       +/-   ##
===========================================
+ Coverage   69.67%   70.90%    +1.23%     
===========================================
  Files        1614       35     -1579     
  Lines      216793     9214   -207579     
===========================================
- Hits       151044     6533   -144511     
+ Misses      65749     2681    -63068     

☔ View full report in Codecov by Harness.
📢 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.