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.

Sign up for GitHub

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

Jump to bottom

Txpool TTL #2217

Merged
merged 9 commits into from
Sep 23, 2024
Merged

Txpool TTL #2217

merged 9 commits into from
Sep 23, 2024

Conversation

AurelienFT
Copy link
Contributor

@AurelienFT AurelienFT commented Sep 18, 2024 •
edited
Loading

REVIEWER NOTE: Review #2216 first for context

Description

Add the TTL management for transactions

Checklist

  • Breaking changes are clearly marked as such in the PR description and changelog
  • New behavior is reflected in tests
  • The specification matches the implemented behavior (link update PR if changes are needed)

Before requesting review

  • I have reviewed the code myself
  • I have created follow-up issues caused by this PR and linked them here

@AurelienFT AurelienFT self-assigned this Sep 18, 2024
@AurelienFT AurelienFT marked this pull request as ready for review September 19, 2024 12:59
@AurelienFT AurelienFT marked this pull request as draft September 19, 2024 13:39
@AurelienFT AurelienFT marked this pull request as ready for review September 20, 2024 08:52
@AurelienFT AurelienFT changed the title Txpool TTL and new limits Txpool TTL Sep 23, 2024
@AurelienFT AurelienFT merged commit 8ed3a5b into txpool-v2-block-producer-importer Sep 23, 2024
9 checks passed
@AurelienFT AurelienFT deleted the txpool-add-limits-and-ttl branch September 23, 2024 22:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xgreenx xgreenx Awaiting requested review from xgreenx

@Dentosal Dentosal Awaiting requested review from Dentosal

@MitchTurner MitchTurner Awaiting requested review from MitchTurner

Assignees

@AurelienFT AurelienFT

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@AurelienFT