mirrors/tendermint
1
0
Fork 0
mirror of https://github.com/tendermint/tendermint.git synced 2026-08-28 20:06:24 +00:00
Code Issues Packages Projects Releases Wiki Activity

cosmetic fixes

Browse Source
This commit is contained in:
William Banfield
2022-02-10 14:54:20 -05:00
parent a2a326f5b7
commit 066a1a845b
2 changed files with 8 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
docs/rfc/README.md
+1
View File
@@ -48,5 +48,6 @@ sections.
- [RFC-008: Don't Panic](./rfc-008-don't-panic.md)
- [RFC-009: Consensus Parameter Upgrades](./rfc-009-consensus-parameter-upgrades.md)
- [RFC-010: P2P Light Client](./rfc-010-p2p-light-client.rst)
- [RFC-011: Delete Gas](./rfc-011-delete-gas.md)
<!-- - [RFC-NNN: Title](./rfc-NNN-title.md) -->
docs/rfc/rfc-011-delete-gas.md
+7 -5
View File
@@ -3,6 +3,7 @@
## Changelog
- 03-Feb-2022: Initial draft (@williambanfield).
- 10-Feb-2022: Update in response to feedback (@williambanfield).
## Abstract
@@ -48,12 +49,13 @@ from the mempool. Overflowing MaxGas is the _only_ way that a transaction can be
invalid that is not directly a result of failing the `CheckTx`. Operators, and the application,
do not know that a transaction was removed from the mempool for this reason. A stateless check
of this nature is exactly what `CheckTx` exists for and there is no reason for the mempool
to keep track of this data separately. A special [MempoolError][add-mempool-error] field was added in v0.35 to communicate
to clients that a transaction failed after `CheckTx`. While this should alleviate the pain for
operators wishing to understand if their transaction was included in the mempool, it
highlights that the abstraction of what is included in the mempool is not currently well defined.
to keep track of this data separately. A special [MempoolError][add-mempool-error] field
was added in v0.35 to communicate to clients that a transaction failed after `CheckTx`.
While this should alleviate the pain for operators wishing to understand if their
transaction was included in the mempool, it highlights that the abstraction of
what is included in the mempool is not currently well defined.
Removing Gas from Tendermint and the mempool would allow for the 'mempool' to be a better
Removing Gas from Tendermint and the mempool would allow for the mempool to be a better
abstraction: any transaction that arrived at `CheckTx` and passed the check will either be
a candidate for a later block or evicted after a TTL is reached. All other transactions are
completely invalid and can be discarded forever.