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

Partial Merit Removals make no sense and should be removed. #317

Open
kayabaNerve opened this issue May 30, 2021 · 0 comments
Open

Partial Merit Removals make no sense and should be removed. #317

kayabaNerve opened this issue May 30, 2021 · 0 comments
Labels
optimization This could be better protocol The protocol has an issue

Comments

@kayabaNerve
Copy link
Member

If the first Element isn't known, as the peer has yet to sync its Block, the PMR will be rejected. If the first Element is known, sending the second Element alone would be enough to trigger the PMR. If the first Element isn't known, the second Element alone may still be accepted instead of rejected, increasing the chances the peer will eventually create the MR.

@kayabaNerve kayabaNerve added protocol The protocol has an issue optimization This could be better labels May 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
optimization This could be better protocol The protocol has an issue
Development

No branches or pull requests

1 participant