-
Notifications
You must be signed in to change notification settings - Fork 52
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
doc: added deletion overview with very high level chart. #288 #295
base: main
Are you sure you want to change the base?
Conversation
website/static/img/Repear-repear.png
Outdated
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.
This is not correct.
OBSOLETE replicas are ALWAYS removed, greedy or not.
The replicas which are removed (in both branches) are the ones with a tombstone.
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.
Also please change repear-repear
to Reaper
All the comments has been resolved !! |
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.
Just one more change :-)
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.
Can you make the bottom left box a little larger? The a., b., c., tangent-ing with the box edge makes it harder to read.
website/static/img/Repear-repear.png
Outdated
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.
Spelling: obsolete not obsolote, and Reaper not repear (see Martin's above comment). The two triangle boxes should be enlarged to make sure the text doesn't escape them.
Added the deletion overveiw in different chapter.
Few things I am not clear.
When setting TOMBSTONE by cleaner and undertaker , what is the value it sets. I look through the code and not clear to me.
The chart are very crude and may need some imporvement suggestion needed here.