Skip to content
This repository has been archived by the owner on Oct 7, 2020. It is now read-only.

Enable stateful CheckTx #9

Closed
ebuchman opened this issue Dec 14, 2016 · 2 comments
Closed

Enable stateful CheckTx #9

ebuchman opened this issue Dec 14, 2016 · 2 comments
Assignees

Comments

@ebuchman
Copy link
Member

See #4

@adrianbrink
Copy link

adrianbrink commented Apr 26, 2017

  1. Change AppendTx to call ApplyTransaction and return the correct information. This will support usage where nonces increment quickly, e.g. multiple txs per block from the same address.
  2. In the future, make CheckTx better by mirroring logic in promoteExecutables() to work for each tx. And also create a new ManagedState upon ABCI Commit(). This will make CheckTx return correct results in the mempool.
  3. Make CheckTx also return address & nonce information and allow tendermint to handle promoting txs that have become valid.

Copied the remaining points from #4.

@adrianbrink adrianbrink changed the title Smarter Mempool Behaviour Enable smarter mempool behaviour May 15, 2017
@adrianbrink adrianbrink added this to the 0.3.0 milestone May 15, 2017
@adrianbrink adrianbrink modified the milestones: 0.5.0, 0.3.0 Jul 23, 2017
@adrianbrink adrianbrink changed the title Enable smarter mempool behaviour Enable stateful CheckTx Aug 14, 2017
@adrianbrink adrianbrink self-assigned this Aug 20, 2017
@zramsay
Copy link
Contributor

zramsay commented Oct 24, 2017

#276

@zramsay zramsay closed this as completed Oct 24, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants