-
Notifications
You must be signed in to change notification settings - Fork 111
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
refactor: extract trin-execution/e2store + clean dependencies #1326
Conversation
0855667
to
4476f6f
Compare
e2store/README.md
Outdated
|
||
## What is era1 | ||
|
||
era1 is a format for storing all of Ethereum's post merge blocks. It contains block headers, block bodies, and receipts for block 0-15537394 |
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.
nit: Should this be pre-merge?
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 meant in the first sentence:
era1 is a format for storing all of Ethereum's post merge blocks
Should this be "post merge" or "pre merge"?
trin-execution/Cargo.toml
Outdated
surf = { version = "2.3.2", default-features = false, features = ["h1-client-rustls", "middleware-logger", "encoding"] } # we use rustls because OpenSSL cause issues compiling on aarch64 | ||
test-log = { version = "0.2.11", features = ["trace"] } | ||
tokio = { version = "1.14.0", features = ["full"] } | ||
tracing-test = "0.1" |
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.
nit: Is there newline at the end? I think it should be.
trin-execution/tests/state.rs
Outdated
const ERA1_FILE_COUNT: usize = 1897; | ||
|
||
/// Fetches era1 files hosted on era1.ethportal.net and shuffles them | ||
async fn get_shuffled_era1_files(http_client: &Client) -> anyhow::Result<Vec<String>> { |
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.
nit: maybe move this into e2store
? Something like test_utils?
What was wrong?
portal-bridge
types, but was rapidly outgrowing its position here.How was it fixed?