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

feat: Remove rules of special reserve blocks #1119

Merged
merged 1 commit into from
Jun 27, 2019
Merged

feat: Remove rules of special reserve blocks #1119

merged 1 commit into from
Jun 27, 2019

Conversation

doitian
Copy link
Member

@doitian doitian commented Jun 26, 2019

For block 1~11, the reward target is genesis block. Yes the reward for genesis is issued multiple times.

Genesis block must have the lock serialized in the cellbase witness, which is set to bootstrap_lock.

@doitian doitian requested a review from a team June 26, 2019 04:09
@nervos-bot
Copy link

nervos-bot bot commented Jun 26, 2019

@quake is assigned as the chief reviewer

@doitian doitian changed the title feat: only genesis has no reward target feat: Remove rules of special reserve blocks Jun 26, 2019
(shared, chain1, chain2)
// the second chain must have smaller hash
if chain1.last().unwrap().header().hash() < chain2.last().unwrap().header().hash() {
(shared, chain2, chain1)
Copy link
Contributor

@u2 u2 Jun 26, 2019

Choose a reason for hiding this comment

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

👍

It's better to add some comments for this function, not in this pr.

For block 1~11, the reward target is genesis block. Genesis block must have the lock serialized in the cellbase witness, which is set to `bootstrap_lock`.
@doitian doitian added breaking change The feature breaks consensus, database, message schema or RPC interface. b:consensus Break consensus b:database Break database schema labels Jun 26, 2019
@doitian
Copy link
Member Author

doitian commented Jun 26, 2019

The most important change is in Consensus::finalize_target. Most of the changes are adding witness for the cellbase in the test.

@zhangsoledad zhangsoledad merged commit 681dc54 into nervosnetwork:develop Jun 27, 2019
@doitian doitian mentioned this pull request Jul 4, 2019
1 task
@doitian doitian deleted the reserve-blocks-reward branch August 9, 2019 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
b:consensus Break consensus b:database Break database schema breaking change The feature breaks consensus, database, message schema or RPC interface.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants