Skip to content

mempool: print postCheck error#2762

Merged
ebuchman merged 1 commit intodevelopfrom
anton/mempool-follow-up
Nov 7, 2018
Merged

mempool: print postCheck error#2762
ebuchman merged 1 commit intodevelopfrom
anton/mempool-follow-up

Conversation

@melekes
Copy link
Contributor

@melekes melekes commented Nov 6, 2018

This is a follow-up from #2724

Closes #2761

  • Updated all relevant documentation in docs
  • Updated all code comments where relevant
  • Wrote tests
  • Updated CHANGELOG_PENDING.md

This is a follow-up from #2724

Closes #2761
@melekes melekes requested review from ebuchman and xla as code owners November 6, 2018 10:15
@codecov-io
Copy link

Codecov Report

Merging #2762 into develop will decrease coverage by 0.02%.
The diff coverage is 66.66%.

@@             Coverage Diff             @@
##           develop    #2762      +/-   ##
===========================================
- Coverage    62.31%   62.28%   -0.03%     
===========================================
  Files          212      212              
  Lines        17193    17210      +17     
===========================================
+ Hits         10713    10719       +6     
- Misses        5583     5591       +8     
- Partials       897      900       +3
Impacted Files Coverage Δ
mempool/mempool.go 77.81% <66.66%> (-0.96%) ⬇️
libs/clist/clist.go 66.66% <0%> (-1.52%) ⬇️
blockchain/pool.go 65.73% <0%> (-0.7%) ⬇️
config/toml.go 53.19% <0%> (-0.66%) ⬇️
consensus/reactor.go 66.55% <0%> (-0.59%) ⬇️
privval/tcp_server.go 81.42% <0%> (+2.85%) ⬆️
libs/events/events.go 98.05% <0%> (+4.85%) ⬆️
lite/base_verifier.go 76% <0%> (+5.41%) ⬆️

@ebuchman ebuchman merged commit d460df1 into develop Nov 7, 2018
@ebuchman ebuchman deleted the anton/mempool-follow-up branch November 7, 2018 04:23
maxim-levy pushed a commit to maxim-levy/tendermint that referenced this pull request Nov 13, 2018
* 'master' of https://github.com/tendermint/tendermint: (330 commits)
  Release/v0.26.1 (tendermint#2803)
  fix amino overhead computation for Tx (tendermint#2792)
  p2p: re-check after sleeps (tendermint#2664)
  check the result of `ps.peer.Send` before calling `ps.setHasVote` (tendermint#2787)
  p2p: AddressBook requires addresses to have IDs; Do not close conn immediately after sending pex addrs in seed mode (tendermint#2797)
  test AutoFile#Size (happy path)
  [autofile/group] do not panic when checking size
  openFile creates a file if not exist => ErrNotExist is not possible
  use our logger in autofile/group
  Add tests for ValidateBasic methods (tendermint#2754)
  [docs] improve organization of ABCI docs & fix links (tendermint#2749)
  p2p: peer-id -> peer_id (tendermint#2771)
  mempool: print postCheck error (tendermint#2762)
  Fix crypto/merkle ProofOperators.Verify to check bounds on keypath pa… (tendermint#2756)
  Mempool WAL is still created by default in home directory, leads to permission errors (tendermint#2758)
  mempool: ErrPreCheck and more log info (tendermint#2724)
  Release/v0.26.0 (tendermint#2726)
  [ADR] [DRAFT] pubsub 2.0 (tendermint#2532)
  validate reactor messages (tendermint#2711)
  TMHASH is 32 bytes. Closes tendermint#1990 (tendermint#2732)
  ...
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.

3 participants