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.

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

feat(rpc): add a method to remove a transaction from tx-pool #3170

Merged
merged 2 commits into from
Dec 13, 2021

Conversation

yangby-cryptape
Copy link
Collaborator

What problem does this PR solve?

If there are some transactions in tx pool, but we want to delete them, at present, the only way is to clear the whole tx pool.

This PR add a new RPC to remove a transaction from tx pool via its hash.

Check List

Tests

  • Integration test

Release note

Title Only: Include only the PR title in the release note.

@yangby-cryptape yangby-cryptape marked this pull request as ready for review November 9, 2021 09:57
@yangby-cryptape yangby-cryptape requested a review from a team as a code owner November 9, 2021 09:57
Copy link
Contributor

@chanhsu001 chanhsu001 left a comment

Choose a reason for hiding this comment

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

the RPC mainly serves "clear some transaction(big cycle transaction, etc.)" and "single node test"

@yangby-cryptape
Copy link
Collaborator Author

the RPC mainly serves "clear some transaction(big cycle transaction, etc.)" and "single node test"

This RPC only provides a new operation, how to use this RPC based on people's imagination.

There is no limit.

tx-pool/src/pool.rs Outdated Show resolved Hide resolved
@yangby-cryptape yangby-cryptape added the s:hold Status: Put this issue on hold. label Nov 26, 2021
@yangby-cryptape yangby-cryptape added s:waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. and removed s:waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. s:hold Status: Put this issue on hold. labels Nov 29, 2021
@zhangsoledad
Copy link
Member

bors r= quake,zhangsoledad

bors bot added a commit that referenced this pull request Dec 5, 2021
3170: feat(rpc): add a method to remove a transaction from tx-pool r=quake,zhangsoledad a=yangby-cryptape

### What problem does this PR solve?

If there are some transactions in tx pool, but we want to delete them, at present, the only way is to clear the whole tx pool.

This PR add a new RPC to remove a transaction from tx pool via its hash.

### Check List

Tests

- Integration test

### Release note

```release-note
Title Only: Include only the PR title in the release note.
```



Co-authored-by: Boyu Yang <yangby@cryptape.com>
@bors
Copy link
Contributor

bors bot commented Dec 5, 2021

Build failed:

@zhangsoledad zhangsoledad added the s:waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. label Dec 5, 2021
@yangby-cryptape yangby-cryptape added s:blocked Status: Blocked and removed s:waiting-on-author Status: The marked PR is awaiting some action (such as code changes) from the PR author. labels Dec 6, 2021
@yangby-cryptape yangby-cryptape removed the s:blocked Status: Blocked label Dec 13, 2021
@yangby-cryptape
Copy link
Collaborator Author

bors r= quake,zhangsoledad

@bors bors bot merged commit 1595009 into nervosnetwork:develop Dec 13, 2021
@yangby-cryptape yangby-cryptape deleted the pr/new-rpc branch December 13, 2021 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants