-
Notifications
You must be signed in to change notification settings - Fork 72
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
Changes to support new block_header_state
and block_state
#1975
Changes from all commits
61ff571
12e3148
03a226d
9bb5575
c38f1a4
6d862bf
3fbced5
bc025d1
bb6e688
5a92916
e94d146
eb8ad28
84f3b85
c55909f
47660e1
3b411a8
cb30d5b
27439fa
c60ded4
cadcb05
640b3ec
ec1f198
f324575
b23ab79
e85b0bf
331ee19
7db4760
b6aa4a2
bbf843f
053c756
8cc1b51
0c210c3
7bd4a0e
6397a0c
c521919
6d70dbe
03db746
5bcc210
a3e1abe
70cb2c3
c2cce4b
3269766
0159cb2
42b2a45
88092c3
60555a4
fffbe1e
b5c6cc0
eddea59
c0804e8
cb83d8f
2ba8a1a
e012dc1
bc2ff35
1258af4
26c61be
d5c64a3
e90f9a1
65df8b3
1bda5c1
ef5f032
de03c87
617b5b7
ab5fd63
d54d3e9
956c08d
d508795
882bb5f
08730c6
c830bea
1ac7744
49fcbae
4a648dc
e535015
6eda25a
6207985
23c9862
5e6be9d
6473bc8
ea199c5
346435b
8d19355
96c029d
32b1d2e
e1295ae
2af37fc
029d5d7
c17ecf9
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,27 @@ | ||
The following diagram describes Leap block production, as implemented in `libraries/chain/controller.cpp`: | ||
|
||
```mermaid | ||
flowchart TD | ||
pp[producer_plugin] --> D | ||
A("replay()"):::fun --> B("replay_push_block()"):::fun | ||
B --> E("maybe_switch_forks()"):::fun | ||
C("init()"):::fun ---> E | ||
C --> A | ||
D("push_block()"):::fun ---> E | ||
subgraph G["apply_block()"] | ||
direction TB | ||
start -- "stage = Ø" --> sb | ||
sb("start_block()"):::fun -- "stage = building_block" --> et | ||
et["execute transactions" ] -- "stage = building_block" --> fb("finalize_block()"):::fun | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Does There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I'm not understanding exactly what you are asking. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
yes, it is. |
||
fb -- "stage = assembled block" --> cb["add transaction metadata and create completed block"] | ||
cb -- "stage = completed block" --> commit("commit_block() (where we [maybe] add to fork_db and mark valid)"):::fun | ||
|
||
end | ||
B ----> start | ||
E --> G | ||
D --> F("log_irreversible()"):::fun | ||
commit -- "stage = Ø" --> F | ||
F -- "if in irreversible mode" --> G | ||
|
||
classDef fun fill:#f96 | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file probably should be located inside /docs/01_nodeos. But you can move it later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I followed @ericpassmore 's suggestion for the file path, but happy to move it if desirable.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no changes for now.