Skip to content

Commit

Permalink
chore: discard abci responses by default (#2764)
Browse files Browse the repository at this point in the history
## Overview

sets the default to discard abci responses

closes #

## Checklist

- [x] New and updated code has appropriate documentation
- [x] New and updated code has new and/or updated testing
- [x] Required CI checks are passing
- [x] Visual proof for any user facing features like CLI or
documentation updates
- [x] Linked issues closed with keywords

(cherry picked from commit 3677fa9)
  • Loading branch information
evan-forbes committed Oct 25, 2023
1 parent 1d303d2 commit 8c7e9df
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion app/default_overrides.go
Original file line number Diff line number Diff line change
Expand Up @@ -233,6 +233,7 @@ func DefaultConsensusConfig() *tmcfg.Config {
// TODO: make TimeoutBroadcastTx configurable per https://github.com/celestiaorg/celestia-app/issues/1034
cfg.RPC.TimeoutBroadcastTxCommit = 50 * time.Second
cfg.RPC.MaxBodyBytes = int64(8388608) // 8 MiB

cfg.Mempool.TTLNumBlocks = 5
cfg.Mempool.TTLDuration = time.Duration(cfg.Mempool.TTLNumBlocks) * appconsts.GoalBlockTime
// Given that there is a stateful transaction size check in CheckTx,
Expand All @@ -242,12 +243,15 @@ func DefaultConsensusConfig() *tmcfg.Config {
upperBoundBytes := appconsts.DefaultSquareSizeUpperBound * appconsts.DefaultSquareSizeUpperBound * appconsts.ContinuationSparseShareContentSize
cfg.Mempool.MaxTxBytes = upperBoundBytes
cfg.Mempool.MaxTxsBytes = int64(upperBoundBytes) * cfg.Mempool.TTLNumBlocks

cfg.Mempool.Version = "v1" // prioritized mempool

cfg.Consensus.TimeoutPropose = appconsts.TimeoutPropose
cfg.Consensus.TimeoutCommit = appconsts.TimeoutCommit
cfg.Consensus.SkipTimeoutCommit = false

cfg.TxIndex.Indexer = "null"
cfg.Storage.DiscardABCIResponses = true

return cfg
}

Expand Down

0 comments on commit 8c7e9df

Please sign in to comment.