Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Added client function to delete a recent block #8533

Merged
3 commits merged into from
May 5, 2021
Merged

Added client function to delete a recent block #8533

3 commits merged into from
May 5, 2021

Conversation

arkpar
Copy link
Member

@arkpar arkpar commented Apr 5, 2021

Required in cumulus to discard obsolete blocks that did not make it into the relay chain.

@arkpar arkpar added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Apr 5, 2021
@arkpar arkpar requested a review from bkchr April 5, 2021 20:37
client/db/src/lib.rs Outdated Show resolved Hide resolved
client/db/src/lib.rs Outdated Show resolved Hide resolved
client/state-db/src/noncanonical.rs Show resolved Hide resolved
arkpar and others added 2 commits May 5, 2021 18:21
Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>
@bkchr
Copy link
Member

bkchr commented May 5, 2021

bot merge

@ghost
Copy link

ghost commented May 5, 2021

Trying merge.

@ghost
Copy link

ghost commented May 5, 2021

Bot will approve on the behalf of @bkchr, since they are a team lead, in an attempt to reach the minimum approval count

@ghost ghost merged commit e5437ef into master May 5, 2021
@ghost ghost deleted the a-drop-block branch May 5, 2021 16:16
nazar-pc pushed a commit to autonomys/substrate that referenced this pull request Aug 8, 2021
* Implemented recent block removal

* Apply suggestions from code review

Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>

Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants