-
Notifications
You must be signed in to change notification settings - Fork 95
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
Offline delete execution after orphan block scan #1145
Conversation
Move calculation of vnode IDs to offline_delete side
@@ -149,23 +146,19 @@ handle_manifest({_UUID, | |||
#state{threshold=Threshold} = _State) | |||
when ContentLength < Threshold -> | |||
{0, 0}; | |||
handle_manifest({_UUID, ?MANIFEST{bkey=BKey={Bucket,_}, | |||
handle_manifest({_UUID, ?MANIFEST{bkey=BKey={Bucket, Key}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd recommend using CSBucket
and CSKey
as this will be confusing (originally I wrote this, sorry!)
If you can fix nitpicks above today, I'll wait . |
... wait +1ing. |
I pushed the update, master. |
Output of manual test said:
I thought the code info("Total number of UUIDs that has any orphaned blocks: ~p [count]", [UUIDs]),
info("Total number of orphaned blocks: ~p [count]", [Blocks]),
info("Orphaned Blocks written to ~p", [filename:absname(Filename)]). shows the file to use next, but it didn't appear. Any idea what was wrong? |
I just removed whole data in LevelDB. |
Ah, that's because I deleted bucket record. mmmm..... |
fmm, failing again
(??) |
oh, I need to run ensure script before. |
Hey @borshop |
…lean Offline delete execution after orphan block scan
Merging as @borshop is temporarily unavailable, manually tested at laptop. |
select_gc_bucket
andoffline_delete
scripts to calculate vnode IDsin
offline_delete
sideoffline_delete
for output files fromblock_audit
andensure_orphan_blocks
select_gc_bucket