Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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: op-reth #4377
feat: op-reth #4377
Changes from 250 commits
60b83b9
87909d4
a053c26
a01b9c9
54957c4
3798be1
a84ea91
6470379
6b110d9
4a12a3a
5e046df
ce6731a
a182909
f17dc3c
7edf44d
5b3b524
3321955
738d7da
4dfb9fa
6966735
f2e6bd0
08ed3d3
afb8172
ee59168
19d082d
06e503d
8924c5e
c21a224
7a4536a
d311c2e
2d70e0b
973cc9a
5b4bd07
455d637
bf9901d
e7a6771
9122353
5d7ea82
68b621a
4c00711
b8fd4dd
2a89bcd
d7dd0e2
fc8ebf9
a8978e2
03395dc
3bc22fa
ada47ff
c23408f
394dc00
a956013
8fbba1e
c14c339
122be97
09ff7e4
1decac7
588be43
0619eb9
12c8750
8c546b1
dc9ee70
32c9046
43309e1
c2ea10f
e173828
504273d
95abdc8
dc97883
0977eb7
9cc55d5
a778a50
4e7cdfb
0f2ea91
d4b0fcf
3ce4b83
53b1a18
10b5881
89eba28
982a16f
308c2c0
841accd
6c7882c
d562387
ba309ec
66e9ba0
71836cd
784860c
0b45fb9
7ab8e06
5483c3b
365f784
5aa493f
3f159c5
0a2d948
d3a7871
d1d24a4
4f1b00f
44dee3e
ad8fb13
766076e
2467bc1
d8b575f
9572071
150b9ff
9bb9ade
1a1a57f
a374ea0
47dd6ac
a78518a
ed6fd6a
6dc2aa4
050fb91
07264fe
bc17092
0cd95a9
9275965
3209735
f6b47a1
af8281d
54a1cdc
e7757ff
6fca9cf
ef538f5
f2e1e35
4d4d15f
56aab40
6abad49
91d710e
d044bc7
8f9a748
474b4f4
ce2b28d
75eb78b
0b079b7
f9344e0
055725f
88f4533
53b543c
1559f96
748f1ea
92421ae
4b85a98
d5810ed
d6f7edf
56460a5
d10dcb7
699bed3
0a449d2
d40e986
db68a1a
c0c7346
e19b137
2aa54bc
568eec9
1e771b5
333103f
c645b28
f542a75
39a8997
bef336e
21f551e
c0b7d34
227e472
c84dea0
5d0d294
1a9662d
589b724
ce627e2
6762c34
232f73d
c75b816
aad68f7
4e40abd
e5ee08c
3a0317c
825cc73
c11ec55
aaccff5
a150662
2914165
6051516
00b8c4b
81128ad
b3c4867
6f8139a
b752b9e
e9f1263
28ede82
f5bf065
ea5ab50
497b96e
773d566
0985ed8
04cb5fc
53cf13f
8bab946
f747108
9875971
a776ce8
9139475
e28f740
f740f34
da3a9c5
07b23f6
d082ae3
d3986a6
96548c6
c1c09a1
89a5c53
13d3b14
bb585fb
ef487c0
779ca19
6de2577
23e9440
3190b1f
63cd43d
9c7ab40
0474763
ac050a2
018789f
9be3212
7e87683
45017e6
1e68098
ac96757
fe52763
eeb2c97
3652bca
8cab07b
9423c95
3f9b4e9
b53ed2c
93d5c20
8492578
cd0efc2
af6d37b
05ca268
db54081
619ce45
7482db8
ec97477
32324c9
b2768d0
53e8fc3
6215e45
1d6febf
d261c71
1c4acfd
62f8111
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
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 don't think we need this?
what is this used for?
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 is a no-op feature, specifically for the CI matrix. We added the
ethereum
feature as a switch for CI with and withoutoptimism
enabled.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 see, makes sense imo
let's add s one line note
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.
wdym "not yet functional"?
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.
As in, not yet implemented. For context - the
compute_pending_block
flag was added toop-geth
around the time that Lattice was making OP Craft to enable higher throughput. This flag is not necessary for MVP, so we decided to stub it out for now.In practice, the
compute_pending_block
flag is meant to allow for skipping the pending block computation in the payload job poller. See: https://github.com/ethereum-optimism/op-geth/blob/7b2e04673aaca3c33c78165d28089b4f5c456417/miner/worker.go#L429-L440There 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'd like this as a separate function on
Conf
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.
Can you expand on this a bit? Not sure if you mean inlining the construction of the
payload_job_config
in thePayloadBuilderConfig
trait since that is the unknownConf
type so that thecompute_pending_block
feature dependent config can happen there.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.
should we open separate issue for this?
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.
@mattsse maybe we should call this
EthereumPayloadBuilder
instead of()
which is a bit weird?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.
Happy to have you all create an issue and assign it to me (i'm sure clabby would also be willing to take it on) :) for after this lands