-
Notifications
You must be signed in to change notification settings - Fork 59
Conversation
@@ -330,15 +328,6 @@ txs included in a proposed block. | |||
Must have `MaxGas >= -1`. | |||
If `MaxGas == -1`, no limit is enforced. | |||
|
|||
### BlockParams.TimeIotaMs |
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.
Removed in tendermint/tendermint#5939
If there are cases in your application where a transaction may become invalid in some | ||
future state, you probably want to disable Tendermint's | ||
cache. You can do that by setting `[mempool] cache_size = 0` in the |
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.
This is a go-specific setting, not really a specification.
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
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.
is this ready to be merged?
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.
A few suggestions, but nothing blocking.
Co-authored-by: M. J. Fromberger <fromberger@interchain.io>
This pull requests offers several fixups to the apps.md document. The changes are primarily in areas that I found somewhat unclear or where language needed to be modified to reflect the more current state of the system. An example of this is the recent rename of fast sync to 'block sync'.