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

fix(datanode): mark the extent as IsDeleted if it exist in deleteExtents map #3072

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

yansenliang
Copy link
Collaborator

What this PR does / why we need it:
Mark the extent as 'IsDeleted' if it exist in deleteExtents map,no need to add it to deleteExtents.

Which issue this PR fixes:

fixes #

Special notes for your reviewer:

Release note:

Copy link

netlify bot commented Feb 4, 2024

Deploy Preview for cubefs-check ready!

Name Link
🔨 Latest commit bfcfefb
🔍 Latest deploy log https://app.netlify.com/sites/cubefs-check/deploys/6642cc7d4bbcc9000845372d
😎 Deploy Preview https://deploy-preview-3072--cubefs-check.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@yansenliang yansenliang marked this pull request as draft February 4, 2024 06:13
@yansenliang yansenliang marked this pull request as ready for review February 4, 2024 06:14
Copy link

codecov bot commented Feb 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 41.99%. Comparing base (bdffd66) to head (a7bd316).
Report is 29 commits behind head on master.

❗ Current head a7bd316 differs from pull request most recent head bfcfefb. Consider uploading reports for the commit bfcfefb to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3072      +/-   ##
==========================================
+ Coverage   40.49%   41.99%   +1.50%     
==========================================
  Files         603      577      -26     
  Lines      125057   118953    -6104     
==========================================
- Hits        50643    49960     -683     
+ Misses      70051    64722    -5329     
+ Partials     4363     4271      -92     

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

@yansenliang yansenliang changed the title fix(datanode): mark the extent as 'IsDeleted' if it exist in deleteExtents map fix(datanode): mark the extent as IsDeleted if it exist in deleteExtents map Feb 22, 2024
@sejust sejust removed their request for review February 22, 2024 11:06
…nts map

Signed-off-by: yansenliang <liangyansen@oppo.com>
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.

None yet

1 participant