-
Notifications
You must be signed in to change notification settings - Fork 90
feat: add txpool_* related RPC methods
#4544
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
Conversation
…#4398) Signed-off-by: nikolay <n.atanasow94@gmail.com>
…olService` interface (#4405) Signed-off-by: nikolay <n.atanasow94@gmail.com>
Signed-off-by: nikolay <n.atanasow94@gmail.com> Signed-off-by: Simeon Nakov <simeon.nakov@limechain.tech> Co-authored-by: Simeon Nakov <simeon.nakov@limechain.tech>
Signed-off-by: nikolay <n.atanasow94@gmail.com> Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com> Co-authored-by: Nikolay Atanasow <n.atanasow94@gmail.com>
Signed-off-by: Simeon Nakov <simeon.nakov@limechain.tech>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com> Signed-off-by: nikolay <n.atanasow94@gmail.com> Co-authored-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: nikolay <n.atanasow94@gmail.com> Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com> Co-authored-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
# Conflicts: # packages/relay/src/lib/precheck.ts
Signed-off-by: nikolay <n.atanasow94@gmail.com>
Signed-off-by: nikolay <n.atanasow94@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
…dded a new log Signed-off-by: Simeon Nakov <simeon.nakov@limechain.tech>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Simeon Nakov <simeon.nakov@limechain.tech>
The base branch was changed.
# Conflicts: # .env.http.example # .env.ws.example # docs/configuration.md # packages/config-service/src/services/globalConfig.ts # packages/relay/src/lib/relay.ts # packages/relay/src/lib/services/ethService/transactionService/TransactionService.ts # packages/relay/src/lib/services/transactionPoolService/LocalPendingTransactionStorage.ts # packages/relay/src/lib/services/transactionPoolService/RedisPendingTransactionStorage.ts # packages/relay/src/lib/services/transactionPoolService/transactionPoolService.ts # packages/relay/src/lib/types/transactionPool.ts # packages/relay/tests/lib/openrpc.spec.ts # packages/relay/tests/lib/services/nonceManagement/RedisPendingTransactionStorage.spec.ts # packages/relay/tests/lib/services/transactionPoolService/LocalPendingTransactionStorage.spec.ts # packages/relay/tests/lib/services/transactionPoolService/transactionPoolService.spec.ts
Signed-off-by: nikolay <n.atanasow94@gmail.com>
Signed-off-by: nikolay <n.atanasow94@gmail.com>
Signed-off-by: nikolay <n.atanasow94@gmail.com>
Signed-off-by: nikolay <n.atanasow94@gmail.com>
Signed-off-by: nikolay <n.atanasow94@gmail.com>
packages/relay/src/lib/services/transactionPoolService/RedisPendingTransactionStorage.ts
Outdated
Show resolved
Hide resolved
quiet-node
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM ready to approve after unused imports removal
Signed-off-by: nikolay <n.atanasow94@gmail.com>
quiet-node
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Great work as always!
Codecov Report❌ Patch coverage is
@@ Coverage Diff @@
## main #4544 +/- ##
========================================
Coverage 95.41% 95.42%
========================================
Files 128 129 +1
Lines 20658 20888 +230
Branches 1762 1773 +11
========================================
+ Hits 19711 19932 +221
- Misses 928 936 +8
- Partials 19 20 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 1 file with indirect coverage changes 🚀 New features to boost your workflow:
|
Description
The transaction pool (txpool) is where pending transactions reside before being submitted to a node. Current APIs require full-pool scans or repeated RPC calls to track transactions, creating unnecessary overhead and inconsistent results.
Adding
txpool_*methods will improve observability, developer usability, and operational efficiency. Richer structured data, better filtering, and optimized access patterns will enable faster, more reliable insights into transaction states and mempool health.Tasks:
txpool_contenttxpool_contentFromtxpool_statusPS: more info can be found in the epic #4488 and in the design doc.
Related issue(s)
Fixes #4488
Testing Guide
Changes from original design (optional)
N/A
Additional work needed (optional)
N/A
Checklist