Skip to content

Block producer #119

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Mar 3, 2020
Merged

Block producer #119

merged 1 commit into from
Mar 3, 2020

Conversation

sergkaprovich
Copy link
Contributor

@sergkaprovich sergkaprovich commented Feb 26, 2020 •
edited
Loading

Description of the Change

Block producer

  • Will be used by Actor Miner
  • Generating Block header, based on parent Tipset and included messages
  • Block header is generating without signature, it must be signed by Actor Miner

Benefits

Generating new blocks for current tipset

@sergkaprovich sergkaprovich force-pushed the feature/chain-block-production branch from 942398c to c884223 Compare February 26, 2020 10:34
@sergkaprovich sergkaprovich changed the title Chain block production Block producer Feb 26, 2020
@sergkaprovich sergkaprovich force-pushed the feature/chain-block-production branch 3 times, most recently from 9899d80 to 3cac326 Compare February 28, 2020 10:40
masterjedy
masterjedy reviewed Feb 28, 2020
View reviewed changes
Copy link
Contributor

@masterjedy masterjedy left a comment

Choose a reason for hiding this comment

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

pls answer the questions and fix minor issues

@sergkaprovich sergkaprovich marked this pull request as ready for review March 2, 2020 10:33
@sergkaprovich sergkaprovich force-pushed the feature/chain-block-production branch from ea101e7 to 6e1a76e Compare March 2, 2020 10:35
@sergkaprovich sergkaprovich requested a review from masterjedy March 2, 2020 15:28
turuslan
turuslan reviewed Mar 3, 2020
View reviewed changes
turuslan
turuslan reviewed Mar 3, 2020
View reviewed changes
turuslan
turuslan reviewed Mar 3, 2020
View reviewed changes
turuslan
turuslan reviewed Mar 3, 2020
View reviewed changes
turuslan
turuslan reviewed Mar 3, 2020
View reviewed changes
turuslan
turuslan reviewed Mar 3, 2020
View reviewed changes
turuslan
turuslan reviewed Mar 3, 2020
View reviewed changes
turuslan
turuslan reviewed Mar 3, 2020
View reviewed changes
turuslan
turuslan reviewed Mar 3, 2020
View reviewed changes
turuslan
turuslan reviewed Mar 3, 2020
View reviewed changes
turuslan
turuslan reviewed Mar 3, 2020
View reviewed changes
turuslan
turuslan reviewed Mar 3, 2020
View reviewed changes
@sergkaprovich sergkaprovich requested a review from turuslan March 3, 2020 09:42
@sergkaprovich sergkaprovich force-pushed the feature/chain-block-production branch from a85c50d to be23e1f Compare March 3, 2020 10:45
@sergkaprovich sergkaprovich merged commit 95e0e04 into master Mar 3, 2020
@sergkaprovich sergkaprovich deleted the feature/chain-block-production branch March 3, 2020 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@turuslan turuslan turuslan approved these changes

@masterjedy masterjedy masterjedy approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sergkaprovich @turuslan @masterjedy