Skip to content

feat: make handshake cancelable (backport #857)#1012

Merged
sergio-mena merged 1 commit intocometbft:v0.34.xfrom
yihuang:v0.34.x
Jun 22, 2023
Merged

feat: make handshake cancelable (backport #857)#1012
sergio-mena merged 1 commit intocometbft:v0.34.xfrom
yihuang:v0.34.x

Conversation

@yihuang
Copy link
Contributor

@yihuang yihuang commented Jun 21, 2023

it'll make the handshake work with graceful shutdown(see: cosmos/cosmos-sdk#16202)

handshake could be a long running process if there are many local blocks to replay, for example we use it to do profiling.


  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments

PR checklist

  • Tests written/updated
  • Changelog entry added in .changelog (we use unclog to manage our changelog)
  • Updated relevant documentation (docs/ or spec/) and code comments

it'll make the handshake work with graceful shutdown(see: cosmos/cosmos-sdk#16202)

handshake could be a long running process if there are many local blocks to replay, for example we use it to do profiling.

Hope we can backport this to 0.34.x.

---

- [ ] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use [unclog](https://github.com/informalsystems/unclog) to manage our changelog)
- [ ] Updated relevant documentation (`docs/` or `spec/`) and code comments
Copy link
Collaborator

@sergio-mena sergio-mena left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this

@sergio-mena sergio-mena self-assigned this Jun 21, 2023
@sergio-mena sergio-mena merged commit 60e431e into cometbft:v0.34.x Jun 22, 2023
@yihuang yihuang deleted the v0.34.x branch June 22, 2023 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants