m3 icon indicating copy to clipboard operation
m3 copied to clipboard

[dbnode] Update WriteShardsInitializing comment

Open ChrisChinchilla opened this issue 4 years ago • 2 comments

The comment for WriteShardsInitializing was a duplicate of the comment for ShardsLeavingCountTowardsConsistency, this PR adds a relevant comment.

ChrisChinchilla avatar Feb 10 '21 16:02 ChrisChinchilla

Codecov Report

Merging #3205 (ee6aa22) into master (0ab7b10) will increase coverage by 14.8%. The diff coverage is n/a.

:exclamation: Current head ee6aa22 differs from pull request most recent head a42eac0. Consider uploading reports for the commit a42eac0 to get more accurate results

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3205      +/-   ##
==========================================
+ Coverage    57.5%    72.3%   +14.8%     
==========================================
  Files         551     1094     +543     
  Lines       64031   100962   +36931     
==========================================
+ Hits        36824    73032   +36208     
+ Misses      24012    22865    -1147     
- Partials     3195     5065    +1870     
Flag Coverage Δ
aggregator 75.8% <ø> (+18.6%) :arrow_up:
cluster 84.9% <ø> (∅)
collector 84.3% <ø> (+25.8%) :arrow_up:
dbnode 78.7% <ø> (+16.3%) :arrow_up:
m3em 74.4% <ø> (+27.9%) :arrow_up:
m3ninx 73.3% <ø> (?)
metrics 20.0% <ø> (+0.2%) :arrow_up:
msg 74.2% <ø> (-0.2%) :arrow_down:
query 67.2% <ø> (?)
x 80.1% <ø> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 0ab7b10...a42eac0. Read the comment docs.

codecov[bot] avatar Feb 10 '21 16:02 codecov[bot]

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Chris Chinchilla seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar May 19 '23 07:05 CLAassistant