Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

It's better read from genDoc than from state.validators when appHeight==0 in replay #2893

Merged
merged 9 commits into from Nov 26, 2018

Conversation

james-ray
Copy link
Contributor

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

fix #2892

@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (develop@72f86b5). Click here to learn what that means.
The diff coverage is 100%.

@@            Coverage Diff             @@
##             develop    #2893   +/-   ##
==========================================
  Coverage           ?   62.18%           
==========================================
  Files              ?      212           
  Lines              ?    17309           
  Branches           ?        0           
==========================================
  Hits               ?    10764           
  Misses             ?     5637           
  Partials           ?      908
Impacted Files Coverage Δ
consensus/replay.go 57.14% <100%> (ø)

@james-ray james-ray changed the title It's better read from genDoc than state.validators when appHeight==0 It's better read from genDoc than state.validators when appHeight==0 in replay Nov 21, 2018
@james-ray james-ray changed the title It's better read from genDoc than state.validators when appHeight==0 in replay It's better read from genDoc than from state.validators when appHeight==0 in replay Nov 21, 2018
Copy link
Contributor

@melekes melekes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🍏

Seems logical to me.

Copy link
Contributor

@ebuchman ebuchman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for catching this!

@ebuchman ebuchman merged commit fe3b97f into tendermint:develop Nov 26, 2018
danil-lashin pushed a commit to danil-lashin/tendermint that referenced this pull request Nov 27, 2018
* develop:
  types: Emit tags from BeginBlock/EndBlock (tendermint#2747)
  Fix fast sync stack with wrong block tendermint#2457 (tendermint#2731)
  It's better read from genDoc than from state.validators when appHeight==0 in replay (tendermint#2893)
  update encoding spec (tendermint#2903)
  return back initially allowed level if we encounter allowed key (tendermint#2889)
  Handling integer IDs in JSON-RPC requests -- fixes tendermint#2366 (tendermint#2811)

# Conflicts:
#	blockchain/reactor_test.go
#	blockchain/store_test.go
#	node/node_test.go
#	types/events.go
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.

None yet

4 participants