Skip to content

add proposed block_import_sequence.svg#622

Closed
haerdib wants to merge 5 commits intomasterfrom
add_block_import_sequence
Closed

add proposed block_import_sequence.svg#622
haerdib wants to merge 5 commits intomasterfrom
add_block_import_sequence

Conversation

@haerdib
Copy link
Contributor

@haerdib haerdib commented Jan 17, 2022

To view file in GitHub:

grafik

Press "View File" and then
grafik

"Raw"

And the picture should be visible. :)


@haerdib haerdib self-assigned this Jan 17, 2022
@haerdib haerdib marked this pull request as ready for review January 17, 2022 16:45
@haerdib haerdib requested review from clangenb and murerfel January 17, 2022 16:45
@haerdib haerdib marked this pull request as draft January 20, 2022 09:15
Copy link
Contributor

@murerfel murerfel left a comment

Choose a reason for hiding this comment

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

I'm afraid the workflow has now changed quite substantially with PR #627 . The sidechain block import now happens inside the block production loop. Gossiped sidechain blocks are just added to a queue. And then at the beginning of every slot (regardless of whether we claim it or not), we process the sidechain import queue.

@haerdib haerdib changed the title add bock_import_sequence.svg add block_import_sequence.svg Jan 27, 2022
@haerdib haerdib changed the title add block_import_sequence.svg add proposed block_import_sequence.svg Jan 27, 2022
@haerdib
Copy link
Contributor Author

haerdib commented Mar 14, 2022

Will be posted into our new wiki, hence closing :)

@haerdib haerdib closed this Mar 14, 2022
@haerdib haerdib deleted the add_block_import_sequence branch July 8, 2022 14:21
@haerdib haerdib restored the add_block_import_sequence branch July 8, 2022 14:21
@clangenb clangenb deleted the add_block_import_sequence branch February 22, 2023 06:59
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.

2 participants