8000 feat(mempool): add error ErrRecheckFull by hvanz · Pull Request #3654 · cometbft/cometbft · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

feat(mempool): add error ErrRecheckFull #3654

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

Merged
merged 1 commit into from
Aug 9, 2024

Conversation

hvanz
Copy link
Member
@hvanz hvanz commented Aug 9, 2024

Complements #3314

This PR adds a new error with a more clear error message when checking if the mempool is full while rechecking is in progress after a new block was committed.


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
  • Title follows the Conventional Commits spec

< 8000 /div>
@hvanz hvanz added the mempool label Aug 9, 2024
@hvanz hvanz self-assigned this Aug 9, 2024
@hvanz hvanz requested review from a team as code owners August 9, 2024 09:55
@hvanz hvanz added backport-to-v1.x Tell Mergify to backport the PR to v1.x backport-to-v0.38.x Tell Mergify to backport the PR to v0.38.x labels Aug 9, 2024
Copy link
Contributor
@melekes melekes left a comment

Choose a reason for hiding this comment

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

👍

@hvanz hvanz added this pull request to the merge queue Aug 9, 2024
Merged via the queue into main with commit 987fa83 Aug 9, 2024
36 checks passed
@hvanz hvanz deleted the hvanz/mempool-add-error-recheck-full branch August 9, 2024 10:58
mergify bot pushed a commit that referenced this pull request Aug 9, 2024
Complements #3314

This PR adds a new error with a more clear error message when checking
if the mempool is full while rechecking is in progress after a new block
was committed.

---

#### PR checklist

- [ ] 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
- [ ] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec

(cherry picked from commit 987fa83)
mergify bot pushed a commit that referenced this pull request Aug 9, 2024
Complements #3314

This PR adds a new error with a more clear error message when checking
if the mempool is full while rechecking is in progress after a new block
was committed.

---

#### PR checklist

- [ ] 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
- [ ] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec

(cherry picked from commit 987fa83)

# Conflicts:
#	mempool/clist_mempool.go
hvanz added a commit that referenced this pull request Aug 9, 2024
Complements #3314

This PR adds a new error with a more clear error message when checking
if the mempool is full while rechecking is in progress after a new block
was committed.

---

#### PR checklist

- [ ] 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
- [ ] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec
<hr>This is an automatic backport of pull request #3654 done by
[Mergify](https://mergify.com).

Co-authored-by: Hernán Vanzetto <15466498+hvanz@users.noreply.github.com>
hvanz added a commit that referenced this pull request Aug 9, 2024
Complements #3314

This PR adds a new error with a more clear error message when checking
if the mempool is full while rechecking is in progress after a new block
was committed.

---

#### PR checklist

- [ ] 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
- [ ] Title follows the [Conventional
Commits](https://www.conventionalcommits.org/en/v1.0.0/) spec
<hr>This is an automatic backport of pull request #3654 done by
[Mergify](https://mergify.com).

---------

Co-authored-by: Hernán Vanzetto <15466498+hvanz@users.noreply.github.com>
Co-authored-by: hvanz <hernan.vanzetto@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-v0.38.x Tell Mergify to backport the PR to v0.38.x backport-to-v1.x Tell Mergify to backport the PR to v1.x mempool
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants
0