-
Notifications
You must be signed in to change notification settings - Fork 65
apollo_l1_provider: remove finality_too_high helper function, add MissingBlock error #10348
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
apollo_l1_provider: remove finality_too_high helper function, add MissingBlock error #10348
Conversation
63b3e91 to
b7ff544
Compare
77f1085 to
159b40f
Compare
159b40f to
2462126
Compare
b7ff544 to
2679f3f
Compare
2462126 to
772a4b0
Compare
2679f3f to
245a3cc
Compare
245a3cc to
bbf0ccb
Compare
772a4b0 to
960a899
Compare
ShahakShama
left a comment
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.
@ShahakShama reviewed 1 of 1 files at r1, all commit messages.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @guy-starkware)
crates/apollo_l1_provider/src/l1_scraper.rs line 261 at r1 (raw file):
.map_err(L1ScraperError::BaseLayerError)?; let latest_l1_block = latest_l1_block
Chain these:
let latest_l1_block = self
.base_layer
.l1_block_at(latest_l1_block_number)
.await
.map_err(L1ScraperError::BaseLayerError)?
.ok_or(L1ScraperError::MissingBlock { block_number: latest_l1_block_number })?;
crates/apollo_l1_provider/src/l1_scraper.rs line 433 at r1 (raw file):
FinalityTooHigh { finality: u64, latest_l1_block_no_finality: L1BlockNumber }, #[error("Block number {block_number} not found")] MissingBlock { block_number: L1BlockNumber },
This error isn't very informative. The fact that we asked for this block is an internal logic of the scraper. Maybe MissingBlockReturnedFromLatestBlockNumber? BaseLayerInconsistency?
guy-starkware
left a comment
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.
Reviewable status: all files reviewed, 2 unresolved discussions (waiting on @ShahakShama)
crates/apollo_l1_provider/src/l1_scraper.rs line 261 at r1 (raw file):
Previously, ShahakShama wrote…
Chain these:
let latest_l1_block = self
.base_layer
.l1_block_at(latest_l1_block_number)
.await
.map_err(L1ScraperError::BaseLayerError)?
.ok_or(L1ScraperError::MissingBlock { block_number: latest_l1_block_number })?;
Done.
crates/apollo_l1_provider/src/l1_scraper.rs line 433 at r1 (raw file):
Previously, ShahakShama wrote…
This error isn't very informative. The fact that we asked for this block is an internal logic of the scraper. Maybe MissingBlockReturnedFromLatestBlockNumber? BaseLayerInconsistency?
LatestL1BlockNumberNoBlockFound
bbf0ccb to
530dd2e
Compare
960a899 to
1990af4
Compare
ShahakShama
left a comment
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.
@ShahakShama reviewed 1 of 1 files at r2, all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @guy-starkware)
|
Artifacts upload workflows: |
guy-starkware
left a comment
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.
@guy-starkware reviewed 1 of 1 files at r2, 4 of 4 files at r3, all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @guy-starkware)
530dd2e to
12063fb
Compare
guy-starkware
left a comment
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.
@guy-starkware reviewed 4 of 4 files at r4, all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @guy-starkware)

No description provided.