Skip to content

feat(mempool): add error ErrRecheckFull#3654

Merged
hvanz merged 1 commit intomainfrom
hvanz/mempool-add-error-recheck-full
Aug 9, 2024
Merged

feat(mempool): add error ErrRecheckFull#3654
hvanz merged 1 commit intomainfrom
hvanz/mempool-add-error-recheck-full

Conversation

@hvanz
Copy link
Collaborator

@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

@hvanz hvanz added the mempool label Aug 9, 2024
@hvanz hvanz self-assigned this Aug 9, 2024
@hvanz hvanz requested a review from a team as a code owner August 9, 2024 09:55
@hvanz hvanz requested a review from a team August 9, 2024 09:55
@hvanz hvanz added backport-to-v1.x backport-to-v0.38.x Tell Mergify to backport the PR to v0.38.x labels Aug 9, 2024
Copy link
Collaborator

@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
@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 mempool

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants